diff options
| author | Tom Lane | 2017-09-01 17:52:54 +0000 |
|---|---|---|
| committer | Tom Lane | 2017-09-01 17:52:54 +0000 |
| commit | bf387028554f45bf0af926525d6db12e7ee4281e (patch) | |
| tree | 68525bd51bd1b414939db256c05a35c66733e728 /src/include/c.h | |
| parent | c7e990bb0f4dfa2fc33e3755a5996da890e5c9cc (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/include/c.h')
| -rw-r--r-- | src/include/c.h | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/src/include/c.h b/src/include/c.h index 0e21f35bc27..576f93f1ebd 100644 --- a/src/include/c.h +++ b/src/include/c.h @@ -316,6 +316,11 @@ typedef unsigned PG_INT128_TYPE uint128; #define PG_INT64_MAX INT64CONST(0x7FFFFFFFFFFFFFFF) #define PG_UINT64_MAX UINT64CONST(0xFFFFFFFFFFFFFFFF) +/* Max value of size_t might also 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 #define HAVE_INT64_TIMESTAMP |