On 03.08.25 22:20, Tom Lane wrote:
It looks like the majority vote is still in favor of writing out
DatumGetPointer instead of using "_D()" functions, so let's roll
with that approach.
I looked through our two versions of the varatt.h changes and
merged them. The attached is only cosmetically different from
yours, I think --- mostly, I kept the comments I'd written.
I've tested this atop 0001-0005 from [1], and it all seems good.
I'd like to move along with getting these changes committed, and
then I'll take another look at the 8-byte-datums-everywhere proposal.
I committed this with the required prerequisite patches. That concludes
this thread, I think. I'll follow up on the remaining work in the
"Datum as struct" thread, and the work in the "8 byte Datums" thread can
also continue.
