Re: libpq: Remove redundant null pointer checks before free() - Mailing list pgsql-hackers

From Michael Paquier
Subject Re: libpq: Remove redundant null pointer checks before free()
Date
Msg-id Yqv0E7iIUfv4cpp2@paquier.xyz
Whole thread Raw
In response to libpq: Remove redundant null pointer checks before free()  (Peter Eisentraut <peter.eisentraut@enterprisedb.com>)
Responses Re: libpq: Remove redundant null pointer checks before free()
Re: libpq: Remove redundant null pointer checks before free()
List pgsql-hackers
On Thu, Jun 16, 2022 at 10:07:33PM +0200, Peter Eisentraut wrote:
> calls, where the "if" part is unnecessary.  This is of course pretty
> harmless, but some functions like scram_free() and freePGconn() have become
> so bulky that it becomes annoying.  So while I was doing some work in that
> area I undertook to simplify this.

Seems fine.  Would some of the buildfarm dinosaurs hiccup on that?
gaur is one that comes into mind.
--
Michael

Attachment

pgsql-hackers by date:

Previous
From: "shiy.fnst@fujitsu.com"
Date:
Subject: RE: Replica Identity check of partition table on subscriber
Next
From: Peter Geoghegan
Date:
Subject: Re: PG 15 (and to a smaller degree 14) regression due to ExprEvalStep size