On 17/04/2026 22:21, Heikki Linnakangas wrote:
On 16/04/2026 20:47, Tom Lane wrote:
Heikki Linnakangas <[email protected]> writes:
On 16/04/2026 17:37, Tom Lane wrote:
Not excited about making massive changes for this.
Having all three would be a very localized change in postgres.h.
Sure, but *using* them in a consistent way would be invasive.
I remain far less certain than Peter is that this discussion has
anything to do with why Coverity is complaining about
ginExtractEntries. I still think we should make some minimum-effort
change to see if the complaint goes away before expending a lot of
brain cells on choosing a final fix.
I think I'm going to commit my proposal to turn PointerGetDatum() back
into a macro, and see if that makes Coverity happy. Then we'll know, and
we can decide on the next steps. Any objections?
WFM.
...
So, pushed a commit that changes just PointerGetDatum() itself, leaving
all those others alone.
As we thought, this made the Coverity warning go away.
I'm happy with the status quo in master, but if we want to introduce new
ConstPointerGetDatum() or NonConstPointerGetDatum() variants instead of
the macro, now is the time to do it.
For backbranches, IMHO we should go with the macro. It's a little scary
to replace such a widely used function as PointerGetDatum() in
back-branches, but I do think this should be fixed. Introducing new
variants doesn't seems even less backpatchable.
- Heikki
