diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2017-09-01 13:52:54 -0400 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2017-09-01 13:52:54 -0400 |
| commit | 074985b26a434f0d3b5c4724834f716b3a480e17 (patch) | |
| tree | 13d1faf6b36a40cb74997a9e2d33ecaec1acf8b5 /src | |
| parent | 1e6c3626063571443a828e5fd2e59d9d8b9f0e91 (diff) | |
Ensure SIZE_MAX can be used throughout our code.
Pre-C99 platforms may lack <stdint.h> and thereby SIZE_MAX. We have
a couple of places using the hack "(size_t) -1" as a fallback, but
it wasn't universally available; which means the code added in commit
2e70d6b5e fails to compile everywhere. Move that hack to c.h so that
we can rely on having SIZE_MAX everywhere.
Per discussion, it'd be a good idea to make the macro's value safe
for use in #if-tests, but that will take a bit more work. This is
just a quick expedient to get the buildfarm green again.
Back-patch to all supported branches, like the previous commit.
Discussion: https://postgr.es/m/15883.1504278595@sss.pgh.pa.us
Diffstat (limited to 'src')
| -rw-r--r-- | src/include/c.h | 4 | ||||
| -rw-r--r-- | src/timezone/private.h | 4 |
2 files changed, 4 insertions, 4 deletions
diff --git a/src/include/c.h b/src/include/c.h index 58b75440c13..47c5e8db112 100644 --- a/src/include/c.h +++ b/src/include/c.h @@ -295,6 +295,10 @@ typedef unsigned long long int uint64; #define UINT64CONST(x) ((uint64) x) #endif +/* Max value of size_t might be missing if we don't have stdint.h */ +#ifndef SIZE_MAX +#define SIZE_MAX ((size_t) -1) +#endif /* Select timestamp representation (float8 or int64) */ #ifdef USE_INTEGER_DATETIMES diff --git a/src/timezone/private.h b/src/timezone/private.h index f031b17b7ef..f78053660e8 100644 --- a/src/timezone/private.h +++ b/src/timezone/private.h @@ -48,10 +48,6 @@ /* Unlike <ctype.h>'s isdigit, this also works if c < 0 | c > UCHAR_MAX. */ #define is_digit(c) ((unsigned)(c) - '0' <= 9) -#ifndef SIZE_MAX -#define SIZE_MAX ((size_t) -1) -#endif - /* * SunOS 4.1.1 libraries lack remove. */ |