diff options
author | Tom Lane <tgl@sss.pgh.pa.us> | 2018-05-23 14:19:04 -0400 |
---|---|---|
committer | Tom Lane <tgl@sss.pgh.pa.us> | 2018-05-23 14:19:04 -0400 |
commit | b929614f5e867c70721b3db31d3dec6cb35e1eb5 (patch) | |
tree | d200845ea01bbedd5ecf8e4fee130250d8232d77 /src | |
parent | 1d96c1b91a4b7da6288ee63671a234b557ff5ccf (diff) | |
download | postgresql-b929614f5e867c70721b3db31d3dec6cb35e1eb5.tar.gz postgresql-b929614f5e867c70721b3db31d3dec6cb35e1eb5.zip |
Remove configure's check for nonstandard "long long" printf modifiers.
We used to claim to support platforms using 'q' or 'I64' as the printf
length modifier for long long int, by dint of replacing snprintf with
our own code which uses the C99 standard 'll' modifier. But that is
only adequate if we use INT64_MODIFIER only in snprintf-based calls,
not directly with the platform's native printf or fprintf. Which
hasn't been the case for years. We had not noticed, partially because
of inadequate test coverage, and partially because the buildfarm is
almost completely bare of machines that won't take 'll'. The last
one seems to have been frogmouth, which was adjusted recently so that
it will take 'll'. We might as well just give up on the pretense
that anything else works, and save ourselves some configure cycles.
Discussion: https://postgr.es/m/13103.1526749980@sss.pgh.pa.us
Discussion: https://postgr.es/m/24769.1526772680@sss.pgh.pa.us
Diffstat (limited to 'src')
-rw-r--r-- | src/include/pg_config.h.in | 2 | ||||
-rw-r--r-- | src/include/pg_config.h.win32 | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/src/include/pg_config.h.in b/src/include/pg_config.h.in index c3320f2c961..89b88042512 100644 --- a/src/include/pg_config.h.in +++ b/src/include/pg_config.h.in @@ -756,7 +756,7 @@ /* Define to 1 if you have the `__strtoull' function. */ #undef HAVE___STRTOULL -/* Define to the appropriate snprintf length modifier for 64-bit ints. */ +/* Define to the appropriate printf length modifier for 64-bit ints. */ #undef INT64_MODIFIER /* Define to 1 if `locale_t' requires <xlocale.h>. */ diff --git a/src/include/pg_config.h.win32 b/src/include/pg_config.h.win32 index e7c45844bd2..2c701fa718c 100644 --- a/src/include/pg_config.h.win32 +++ b/src/include/pg_config.h.win32 @@ -579,7 +579,7 @@ /* Define to 1 if your compiler understands __VA_ARGS__ in macros. */ #define HAVE__VA_ARGS 1 -/* Define to the appropriate snprintf length modifier for 64-bit ints. */ +/* Define to the appropriate printf length modifier for 64-bit ints. */ #define INT64_MODIFIER "ll" /* Define to 1 if `locale_t' requires <xlocale.h>. */ |