Re: Consistently use macro HeapTupleIsValid to check the validity of tuples in tablecmds.c
Michael Paquier <michael@paquier.xyz>
From: Michael Paquier <michael@paquier.xyz>
To: Heikki Linnakangas <hlinnaka@iki.fi>
Cc: Tom Lane <tgl@sss.pgh.pa.us>, Tender Wang <tndrwang@gmail.com>, PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Date: 2025-04-10T02:53:36Z
Lists: pgsql-hackers
On Wed, Apr 09, 2025 at 05:43:24PM +0300, Heikki Linnakangas wrote: > Agreed. I use both, depending on which mood I'm in. Same here, extended to OidIsValid(), HeapTupleIsValid(), XLogRecPtr, etc., and I tend to prefer such macros, except if consistency of the surroundings matter most. FWIW, I think that living with the current state of things to limit backpatch pain is fine. There is no need to change the existing code as an attempt to apply more standardization even if one or more code grammar patterns mean the same thing. -- Michael
Commits
-
Remove PointerIsValid()
- a5b35fcedb54 19 (unreleased) landed