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
index af799dc1dfb3f849f18aa7c43c7120d12fc73001..4f8bbfcd621fcb96f7efa60f9645f506d851c3aa 100644 (file)
#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
+
/*
* We now always use int64 timestamps, but keep this symbol defined for the
* benefit of external code that might test it.
index c55334906689bd35752f3bd5babf67cca0a64af5..869c59dc8535bd47641b69499bab556bd780f36c 100644 (file)
#define AllocSizeIsValid(size) ((Size) (size) <= MaxAllocSize)
-#define MaxAllocHugeSize ((Size) -1 >> 1) /* SIZE_MAX / 2 */
+#define MaxAllocHugeSize (SIZE_MAX / 2)
#define AllocHugeSizeIsValid(size) ((Size) (size) <= MaxAllocHugeSize)
index c141fb61314501629ddbfd17c2f48ab2e7ba617e..e65cd1bb4ecd29a91afddd9239319da3e40917d2 100644 (file)
/* 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.
*/