pgsql: Fix style violations in syscache lookups.

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: Fix style violations in syscache lookups.
Date: 2019-05-05 17:10:15
Message-ID: E1hNKeV-00008W-Vo@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Fix style violations in syscache lookups.

Project style is to check the success of SearchSysCacheN and friends
by applying HeapTupleIsValid to the result. A tiny minority of calls
creatively did it differently. Bring them into line with the rest.

This is just cosmetic, since HeapTupleIsValid is indeed just a null
check at the moment ... but that may not be true forever, and in any
case it puts a mental burden on readers who may wonder why these
call sites are not like the rest.

Back-patch to v11 just to keep the branches in sync. (The bulk of these
errors seem to have originated in v11 or v12, though a few are old.)

Per searching to see if anyplace else had made the same error
repaired in 62148c352.

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/9691aa72e2a7fb146ac759e1f8a8b04962128cc0

Modified Files
--------------
src/backend/catalog/partition.c | 2 +-
src/backend/catalog/pg_publication.c | 2 +-
src/backend/commands/indexcmds.c | 2 +-
src/backend/commands/opclasscmds.c | 4 ++--
src/backend/commands/operatorcmds.c | 2 +-
src/backend/commands/statscmds.c | 2 +-
src/backend/commands/tablecmds.c | 16 +++++++---------
src/backend/optimizer/util/appendinfo.c | 2 +-
src/backend/optimizer/util/plancat.c | 2 +-
9 files changed, 16 insertions(+), 18 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2019-05-05 17:10:16 pgsql: Fix style violations in syscache lookups.
Previous Message Tomas Vondra 2019-05-04 20:04:37 Re: pgsql: Fix union for pgstat message types