From: Tom Lane <tgl@sss.pgh.pa.us>
To: pgsql-committers@lists.postgresql.org
Subject: pgsql: Mop-up for Datum conversion cleanups.
Date: Fri, 08 Aug 2025 22:45:04 +0000
Message-ID: <E1ukVpf-001S4U-2F@gemulon.postgresql.org> (raw)
Mop-up for Datum conversion cleanups.
Fix a couple more places where an explicit Datum conversion
is needed (not clear how we missed these in ff89e182d and
previous commits).
Replace the minority usage "(Datum) NULL" with "(Datum) 0".
The former depends on the assumption that Datum is the same
width as Pointer, the latter doesn't. Anyway consistency
is a good thing.
This is, I believe, the last of the notational mop-up needed
before we can consider changing Datum to uint64 everywhere.
It's also important cleanup for more aggressive ideas such
as making Datum a struct.
Discussion: https://postgr.es/m/1749799.1752797397@sss.pgh.pa.us
Discussion: https://postgr.es/m/8246d7ff-f4b7-4363-913e-827dadfeb145@eisentraut.org
Branch
------
master
Details
-------
https://git.postgresql.org/pg/commitdiff/665c3dbba497b795c4ee46145777bc4eb89c78a1
Modified Files
--------------
contrib/ltree/_ltree_gist.c | 2 +-
src/backend/catalog/pg_aggregate.c | 2 +-
src/backend/catalog/pg_constraint.c | 2 +-
src/backend/catalog/pg_conversion.c | 2 +-
src/backend/catalog/pg_namespace.c | 2 +-
src/backend/catalog/pg_operator.c | 4 ++--
src/backend/catalog/pg_type.c | 2 +-
src/backend/executor/spi.c | 2 +-
src/backend/nodes/readfuncs.c | 2 +-
src/backend/utils/adt/jsonfuncs.c | 6 +++---
src/backend/utils/adt/rangetypes.c | 4 ++--
src/include/access/htup_details.h | 2 +-
src/include/access/itup.h | 2 +-
13 files changed, 17 insertions(+), 17 deletions(-)
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: pgsql-committers@postgresql.org
Cc: tgl@sss.pgh.pa.us, pgsql-committers@lists.postgresql.org
Subject: Re: pgsql: Mop-up for Datum conversion cleanups.
In-Reply-To: <E1ukVpf-001S4U-2F@gemulon.postgresql.org>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox