postgres.git / summary / log / commit / refs

commit    0bfcda990405b9774dbe9b7d4ee71fa28a2599b3
Author:   Tom Lane <tgl@sss.pgh.pa.us>
Date:     Fri Sep 01 17:52:54 2017 +0000

    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


 src/include/c.h        | 4 ++++
 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 13e6e873799..ebe4b2039b3 100644
--- a/src/include/c.h
+++ b/src/include/c.h
@@ -290,6 +290,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.
  */


[parent: 2a82170a56cb]