pgsql: Add -Wshadow=compatible-local to the standard compilation flags

From: David Rowley <drowley(at)postgresql(dot)org>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: Add -Wshadow=compatible-local to the standard compilation flags
Date: 2022-10-07 03:50:52
Message-ID: E1ogeNv-001FkD-Ug@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Add -Wshadow=compatible-local to the standard compilation flags

Since cd4e8caaa, we've been able to build the source tree with
-Wshadow=compatible-local without any warnings. Lots of work was done by
Justin Pryzby and I (David) to get all our code to compile warning free
with that flag. In that process, 2 bugs (16d69ec29 and af7d270dd) were
discovered and fixed. Additionally, "git log --grep=shadow" shows that
there is no shortage of other bugs that have been fixed over the years
which were caused by variable shadowing.

In light of the above, it seems very much worthwhile to add at least
-Wshadow=compatible-local to our standard compilation flags. We *may*
want to go further and take this to -Wshadow=local in the future, but
we're not ready for that today, so let's add -Wshadow=compatible-local now
to help make sure we don't introduce further local variable shadowing.

Author: Andres Freund
Discussion: https://postgr.es/m/20221006003920.6xlqaoccxwisza5k@awork3.anarazel.de

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/0fe954c28584169938e5c0738cfaa9930ce77577

Modified Files
--------------
configure | 91 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
configure.ac | 2 ++
meson.build | 1 +
3 files changed, 94 insertions(+)

Browse pgsql-committers by date

  From Date Subject
Next Message Peter Eisentraut 2022-10-07 11:38:05 pgsql: Remove unnecessary uses of Abs()
Previous Message Tom Lane 2022-10-07 01:24:12 pgsql: Improve our ability to detect bogus pointers passed to pfree et