Re: perl 5.36, C99, -Wdeclaration-after-statement -Wshadow=compatible-local

Hi,

On 2022-12-29 13:51:37 -0500, Tom Lane wrote:
> We might find that some GCC-impostor compilers have trouble with it,
> but if so we can adjust the #ifdef here.

Yea. I suspect it's widely enough used that any compiler claiming to be gcc
compatible has it, but ...


> Getting nitpicky, I suggest calling it "HAVE_PRAGMA_GCC_SYSTEM_HEADER"
> to align better with what you actually have to write.

Makes sense.


> + * Newer versions the perl headers trigger a lot of warnings with our compiler
> 
> "Newer versions of ..." please.  Otherwise LGTM.

Oops.


> > Should we backpatch this? Given the volume of warnings it's probably a good
> > idea. But I'd let it step in HEAD for a few days of buildfarm coverage first.
> 
> +1 to both points.

Pushed to HEAD.

Greetings,

Andres Freund