database/postgres
1
0
Fork 0
mirror of https://github.com/postgres/postgres.git synced 2025-07-15 19:21:59 +03:00
Code Activity

In messages, use singular nouns for -1, like we do for +1.

Browse Source 
This outputs "-1 year", not "-1 years".

Reported-by: neverov.max@gmail.com

Bug: 16939

Discussion: https://postgr.es/m/16939-cceeb03fb72736ee@postgresql.org
This commit is contained in:
Bruce Momjian
2021-03-30 18:34:27 -04:00
parent 6131ffc43f
commit 5da9868ed9
4 changed files with 17 additions and 17 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
src/backend/utils/adt/datetime.c
View File

@ -4190,7 +4190,7 @@ AddPostgresIntPart(char *cp, int value, const char *units,
(*is_before && value > 0) ? "+" : "", (*is_before && value > 0) ? "+" : "",
value, value,
units, units,
(value != 1) ? "s" : ""); (abs(value) != 1) ? "s" : "");
/* /*
* Each nonzero field sets is_before for (only) the next one. This is a * Each nonzero field sets is_before for (only) the next one. This is a
@ -4216,7 +4216,7 @@ AddVerboseIntPart(char *cp, int value, const char *units,
} }
else if (*is_before) else if (*is_before)
value = -value; value = -value;
sprintf(cp, " %d %s%s", value, units, (value == 1) ? "" : "s"); sprintf(cp, " %d %s%s", value, units, (abs(value) == 1) ? "" : "s");
*is_zero = false; *is_zero = false;
return cp + strlen(cp); return cp + strlen(cp);
} }

4
src/interfaces/ecpg/pgtypeslib/interval.c
View File

@ -694,7 +694,7 @@ AddVerboseIntPart(char *cp, int value, const char *units,
} }
else if (*is_before) else if (*is_before)
value = -value; value = -value;
sprintf(cp, " %d %s%s", value, units, (value == 1) ? "" : "s"); sprintf(cp, " %d %s%s", value, units, (abs(value) == 1) ? "" : "s");
*is_zero = false; *is_zero = false;
return cp + strlen(cp); return cp + strlen(cp);
} }
@ -711,7 +711,7 @@ AddPostgresIntPart(char *cp, int value, const char *units,
(*is_before && value > 0) ? "+" : "", (*is_before && value > 0) ? "+" : "",
value, value,
units, units,
(value != 1) ? "s" : ""); (abs(value) != 1) ? "s" : "");
/* /*
* Each nonzero field sets is_before for (only) the next one. This is a * Each nonzero field sets is_before for (only) the next one. This is a

4
src/interfaces/libpq/fe-print.c
View File

@ -303,7 +303,7 @@ PQprint(FILE *fout, const PGresult *res, const PQprintOpt *po)
} }
if (po->header && !po->html3) if (po->header && !po->html3)
fprintf(fout, "(%d row%s)\n\n", PQntuples(res), fprintf(fout, "(%d row%s)\n\n", PQntuples(res),
(PQntuples(res) == 1) ? "" : "s"); (abs(PQntuples(res)) == 1) ? "" : "s");
if (po->html3 && !po->expanded) if (po->html3 && !po->expanded)
fputs("</table>\n", fout); fputs("</table>\n", fout);
free(fieldMax); free(fieldMax);
@ -662,7 +662,7 @@ PQdisplayTuples(const PGresult *res,
if (!quiet) if (!quiet)
fprintf(fp, "\nQuery returned %d row%s.\n", PQntuples(res), fprintf(fp, "\nQuery returned %d row%s.\n", PQntuples(res),
(PQntuples(res) == 1) ? "" : "s"); (abs(PQntuples(res)) == 1) ? "" : "s");
fflush(fp); fflush(fp);

22
src/test/regress/expected/interval.out
View File

@ -23,15 +23,15 @@ SELECT INTERVAL '-08:00' AS "Eight hours";
(1 row) (1 row)
SELECT INTERVAL '-1 +02:03' AS "22 hours ago..."; SELECT INTERVAL '-1 +02:03' AS "22 hours ago...";
22 hours ago... 22 hours ago...
------------------- ------------------
-1 days +02:03:00 -1 day +02:03:00
(1 row) (1 row)
SELECT INTERVAL '-1 days +02:03' AS "22 hours ago..."; SELECT INTERVAL '-1 days +02:03' AS "22 hours ago...";
22 hours ago... 22 hours ago...
------------------- ------------------
-1 days +02:03:00 -1 day +02:03:00
(1 row) (1 row)
SELECT INTERVAL '1.5 weeks' AS "Ten days twelve hours"; SELECT INTERVAL '1.5 weeks' AS "Ten days twelve hours";
@ -288,7 +288,7 @@ FROM INTERVAL_MULDIV_TBL;
product product
------------------------------------ ------------------------------------
1 year 12 days 122:24:00 1 year 12 days 122:24:00
-1 years -12 days +93:36:00 -1 year -12 days +93:36:00
-3 days -14:24:00 -3 days -14:24:00
2 mons 13 days 01:22:28.8 2 mons 13 days 01:22:28.8
-10 mons +120 days 37:28:21.6567 -10 mons +120 days 37:28:21.6567
@ -317,7 +317,7 @@ FROM INTERVAL_MULDIV_TBL;
---------------------------------- ----------------------------------
4 mons 4 days 40:48:00 4 mons 4 days 40:48:00
-4 mons -4 days +31:12:00 -4 mons -4 days +31:12:00
-1 days -04:48:00 -1 day -04:48:00
25 days -15:32:30.4 25 days -15:32:30.4
-3 mons +30 days 12:29:27.2189 -3 mons +30 days 12:29:27.2189
12 days 12 days
@ -785,9 +785,9 @@ SELECT interval '+1 -1:00:00',
interval '-1 +1:00:00', interval '-1 +1:00:00',
interval '+1-2 -3 +4:05:06.789', interval '+1-2 -3 +4:05:06.789',
interval '-1-2 +3 -4:05:06.789'; interval '-1-2 +3 -4:05:06.789';
interval | interval | interval | interval interval | interval | interval | interval
-----------------+-------------------+-------------------------------------+---------------------------------------- -----------------+------------------+-------------------------------------+---------------------------------------
1 day -01:00:00 | -1 days +01:00:00 | 1 year 2 mons -3 days +04:05:06.789 | -1 years -2 mons +3 days -04:05:06.789 1 day -01:00:00 | -1 day +01:00:00 | 1 year 2 mons -3 days +04:05:06.789 | -1 year -2 mons +3 days -04:05:06.789
(1 row) (1 row)
-- test output of couple non-standard interval values in the sql style -- test output of couple non-standard interval values in the sql style