Re: Remove PointerIsValid()

Peter Geoghegan <pg@bowt.ie>

From: Peter Geoghegan <pg@bowt.ie>
To: Peter Eisentraut <peter@eisentraut.org>
Cc: pgsql-hackers <pgsql-hackers@postgresql.org>
Date: 2025-09-17T14:15:10Z
Lists: pgsql-hackers
On Wed, Sep 17, 2025 at 1:21 AM Peter Eisentraut <peter@eisentraut.org> wrote:
> I think there is agreement that the PointerIsValid() macro is pretty
> useless.  This patch proposes to remove it.  There have been a few
> recent mini-discussions in other threads that appear to support this. [0][1]

I'm also in favor of removing it.

> There were the usual concerns about code churn and backpatching and so
> on, but I think in the end the change is not that big and it's in pretty
> boring positions.

I agree.

-- 
Peter Geoghegan



Commits

  1. Remove PointerIsValid()