pgsql: Adjust DatumGetBool macro, this time for sure. - Mailing list pgsql-committers

From Tom Lane
Subject pgsql: Adjust DatumGetBool macro, this time for sure.
Date
Msg-id E1avoE8-0007v3-5h@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Adjust DatumGetBool macro, this time for sure.

Commit 23a41573c attempted to fix the DatumGetBool macro to ignore bits
in a Datum that are to the left of the actual bool value.  But it did that
by casting the Datum to bool; and on compilers that use C99 semantics for
bool, that ends up being a whole-word test, not a 1-byte test.  This seems
to be the true explanation for contrib/seg failing in VS2015.  To fix, use
GET_1_BYTE() explicitly.  I think in the previous patch, I'd had some idea
of not having to commit to bool being exactly 1 byte wide, but regardless
of what the compiler's bool is, boolean columns and Datums are certainly
1 byte wide.

The previous fix was (eventually) back-patched into all active versions,
so do likewise with this one.

Branch
------
REL9_3_STABLE

Details
-------
http://git.postgresql.org/pg/commitdiff/707c44fe257bc2a7be8acd2d9a072a83e21a60e3

Modified Files
--------------
src/include/postgres.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)


pgsql-committers by date:

Previous
From: Tom Lane
Date:
Subject: pgsql: Adjust DatumGetBool macro, this time for sure.
Next
From: Bruce Momjian
Date:
Subject: Re: pgsql: Fix support of digits in email/hostnames.