From: | David Rowley <drowley(at)postgresql(dot)org> |
---|---|
To: | pgsql-committers(at)lists(dot)postgresql(dot)org |
Subject: | pgsql: Remove shadowed local variables that are new in v15 |
Date: | 2022-08-19 23:42:04 |
Message-ID: | E1oPBcp-000KKL-28@gemulon.postgresql.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
Remove shadowed local variables that are new in v15
Compiling with -Wshadow=compatible-local yields quite a few warnings about
local variables being shadowed by compatible local variables in an inner
scope. Of course, this is perfectly valid in C, but we have had bugs in
the past as a result of developers failing to notice this. af7d270dd is a
recent example.
Here we do a cleanup of warnings we receive from -Wshadow=compatible-local
for code which is new to PostgreSQL 15. We've yet to have the discussion
about if we actually ever want to run that as a standard compilation flag.
We'll need to at least get the number of warnings down to something easier
to manage before we can realistically consider if we want this or not.
This commit is the first step towards reducing the warnings.
The changes being made here are all fairly trivial. Because of that, and
the fact that v15 is still in beta, this is being back-patched into 15.
It seems more risky not to do this as the risk of future bugs is increased
by the additional conflicts that this commit could cause for any future
bug fixes touching the same areas as this commit.
Author: Justin Pryzby
Discussion: https://postgr.es/m/20220817145434.GC26426%40telsasoft.com
Backpatch-through: 15
Branch
------
REL_15_STABLE
Details
-------
https://git.postgresql.org/pg/commitdiff/24f457aa2b7006e4b427a3ac1e7ce0f248b55ba3
Modified Files
--------------
src/backend/backup/basebackup_target.c | 12 ++++-----
src/backend/parser/parse_jsontable.c | 6 ++---
src/backend/replication/logical/tablesync.c | 23 ++++++-----------
src/backend/utils/adt/jsonpath_exec.c | 6 ++---
src/bin/pg_dump/pg_dump.c | 38 ++++++++++++++---------------
5 files changed, 39 insertions(+), 46 deletions(-)
From | Date | Subject | |
---|---|---|---|
Next Message | David Rowley | 2022-08-20 03:18:02 | pgsql: Reduce warnings with -Wshadow=compatible-local builds |
Previous Message | David Rowley | 2022-08-19 23:41:30 | pgsql: Remove shadowed local variables that are new in v15 |