From: Alvaro Herrera <alvherre@alvh.no-ip.org>
To: Andres Freund <andres@anarazel.de>
Cc: Tom Lane <tgl@sss.pgh.pa.us>
Cc: David Rowley <dgrowleyml@gmail.com>
Cc: Justin Pryzby <pryzby@telsasoft.com>
Cc: pgsql-hackers@postgresql.org, Tomas Vondra <tomas.vondra@postgresql.org>
Cc: Peter Smith <smithpb2250@gmail.com>
Subject: Re: shadow variables - pg15 edition
Date: Mon, 10 Oct 2022 18:53:58 +0200
Message-ID: <20221010165358.7aion6kdxpjonpkk@alvherre.pgsql> (raw)
In-Reply-To: <20221010164549.7e5v6vvmcdtxuvov@awork3.anarazel.de>
On 2022-Oct-10, Andres Freund wrote:
> On 2022-10-10 09:37:38 -0700, Andres Freund wrote:
> > On 2022-10-10 18:33:11 +0200, Alvaro Herrera wrote:
> > > On 2022-Oct-10, Andres Freund wrote:
> > >
> > > > Given the age of affected perl instances I suspect there'll not be a lot of
> > > > developers affected, and the number of warnings is reasonably small too. It'd
> > > > likely hurt more developers to not see the warnings locally, given that such
> > > > shadowing often causes bugs.
> > >
> > > Maybe we can install a filter-out in src/pl/plperl's Makefile for the
> > > time being.
> >
> > We could, but is it really a useful thing for something fixed 6 years ago?
Well, for people purposefully installing using older installs of Perl
(not me, admittedly), it does seem useful, because you get the benefit
of checking shadow vars for the rest of the tree and still get no
warnings if everything is clean.
> As an out, a hypothetical dev could add -Wno-shadow=compatible-local to their
> CFLAGS.
But that disables it for the tree as a whole, which is not better.
We can remove the filter-out when we decide to move the Perl version
requirement up, say 4 years from now.
--
Álvaro Herrera 48°01'N 7°57'E — https://www.EnterpriseDB.com/
"El hombre nunca sabe de lo que es capaz hasta que lo intenta" (C. Dickens)
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: pgsql-hackers@postgresql.org
Cc: alvherre@alvh.no-ip.org, andres@anarazel.de, tgl@sss.pgh.pa.us, dgrowleyml@gmail.com, pryzby@telsasoft.com, tomas.vondra@postgresql.org, smithpb2250@gmail.com
Subject: Re: shadow variables - pg15 edition
In-Reply-To: <20221010165358.7aion6kdxpjonpkk@alvherre.pgsql>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox