Re: Enable -Wstrict-prototypes and -Wold-style-definition by default - Mailing list pgsql-hackers

From Peter Eisentraut
Subject Re: Enable -Wstrict-prototypes and -Wold-style-definition by default
Date
Msg-id 4ab608af-2f90-47d8-88a5-6329f6630fff@eisentraut.org
Whole thread Raw
In response to Re: Enable -Wstrict-prototypes and -Wold-style-definition by default  (Peter Eisentraut <peter@eisentraut.org>)
Responses Re: Enable -Wstrict-prototypes and -Wold-style-definition by default
Re: Enable -Wstrict-prototypes and -Wold-style-definition by default
List pgsql-hackers
On 18.03.26 14:32, Peter Eisentraut wrote:
> On 16.03.26 10:55, Peter Eisentraut wrote:
>> On 09.03.26 17:39, Bertrand Drouvot wrote:
>>> 0001: Prevent -Wstrict-prototypes and -Wold-style-definition warnings
>>>
>>> It fixes the remaining warnings that those new flags would generate.
>>
>> I have committed this one.  I'll look at the rest next.
> 
> Also committed.

I have a couple of follow-up patches that I had developed while playing 
with this.

There is a warning option for MSVC that appears to have a very similar 
effect to the ones we added here, so I propose we add that one as well.

Additionally, there is an option for MSVC to disable warnings in system 
headers, similar to the default behavior of GCC.  This would be required 
here because some system header files have non-strict prototypes.

Additionally, I propose to add -Wold-style-declaration, which is 
completely unrelated to -Wold-style-definition, but it has popped up a 
few times via the buildfarm (grep for it in git log), so I think we 
might as well add it so that everyone sees it.

Finally, I think we can remove the option -Wendif-labels, which doesn't 
do anything anymore that isn't the default.  (It was only not the 
default before gcc 4.0.)

Attachment

pgsql-hackers by date:

Previous
From: Nathan Bossart
Date:
Subject: Re: Add RISC-V Zbb popcount optimization
Next
From: Tom Lane
Date:
Subject: Re: Enable -Wstrict-prototypes and -Wold-style-definition by default