shadow variables - pg15 edition

Поиск
Список
Период
Сортировка
От Justin Pryzby
Тема shadow variables - pg15 edition
Дата
Msg-id 20220817145434.GC26426@telsasoft.com
обсуждение исходный текст
Ответы Re: shadow variables - pg15 edition  (Peter Smith <smithpb2250@gmail.com>)
Re: shadow variables - pg15 edition  (David Rowley <dgrowleyml@gmail.com>)
Список pgsql-hackers
There's been no progress on this in the past discussions.

https://www.postgresql.org/message-id/flat/877k1psmpf.fsf%40mailbox.samurai.com

https://www.postgresql.org/message-id/flat/CAApHDvpqBR7u9yzW4yggjG%3DQfN%3DFZsc8Wo2ckokpQtif-%2BiQ2A%40mail.gmail.com#2d900bfe18fce17f97ec1f00800c8e27

https://www.postgresql.org/message-id/flat/MN2PR18MB2927F7B5F690065E1194B258E35D0%40MN2PR18MB2927.namprd18.prod.outlook.com

But an unfortunate consequence of not fixing the historic issues is that it
precludes the possibility that anyone could be expected to notice if they
introduce more instances of the same problem (as in the first half of these
patches).  Then the hole which has already been dug becomes deeper, further
increasing the burden of fixing the historic issues before being able to use
-Wshadow.

The first half of the patches fix shadow variables newly-introduced in v15
(including one of my own patches), the rest are fixing the lowest hanging fruit
of the "short list" from COPT=-Wshadow=compatible-local

I can't see that any of these are bugs, but it seems like a good goal to move
towards allowing use of the -Wshadow* options to help avoid future errors, as
well as cleanliness and readability (rather than allowing it to get harder to
use -Wshadow).

-- 
Justin

Вложения

В списке pgsql-hackers по дате отправления:

Предыдущее
От: "Drouvot, Bertrand"
Дата:
Сообщение: Re: SYSTEM_USER reserved word implementation
Следующее
От: Robert Haas
Дата:
Сообщение: Re: XLogBeginRead's header comment lies