From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | pgsql-committers(at)postgresql(dot)org |
Subject: | pgsql: Ensure SIZE_MAX can be used throughout our code. |
Date: | 2017-09-01 17:53:24 |
Message-ID: | E1dnq8C-0005yg-8J@gemulon.postgresql.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
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
Branch
------
master
Details
-------
https://git.postgresql.org/pg/commitdiff/b79d69b087561eb6687373031a5098b0694f9ec6
Modified Files
--------------
src/include/c.h | 5 +++++
src/include/utils/memutils.h | 2 +-
src/timezone/private.h | 4 ----
3 files changed, 6 insertions(+), 5 deletions(-)
From | Date | Subject | |
---|---|---|---|
Next Message | Peter Eisentraut | 2017-09-01 18:20:44 | pgsql: doc: Remove mentions of server-side CRL and CA file names |
Previous Message | Robert Haas | 2017-09-01 16:32:47 | pgsql: pg_dumpall: Add a -E flag to set the encoding, like pg_dump has. |