pgsql: Mop-up for Datum conversion cleanups.

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: Mop-up for Datum conversion cleanups.
Date: 2025-08-08 22:45:04
Message-ID: E1ukVpf-001S4U-2F@gemulon.postgresql.org
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-committers

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(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Thomas Munro 2025-08-09 01:26:14 pgsql: Fix rare bug in read_stream.c's split IO handling.
Previous Message Peter Eisentraut 2025-08-08 20:11:47 pgsql: Add missing Datum conversions