Re: Should we remove -Wdeclaration-after-statement? - Mailing list pgsql-hackers

From jian he
Subject Re: Should we remove -Wdeclaration-after-statement?
Date
Msg-id CACJufxE9w0hJCwqQ5bAit2mvNc9DKFpD4dNYtg6iH7kG0puO-Q@mail.gmail.com
Whole thread Raw
In response to Re: Should we remove -Wdeclaration-after-statement?  (Jelte Fennema-Nio <postgres@jeltef.nl>)
List pgsql-hackers
On Mon, Jan 29, 2024 at 11:04 PM Jelte Fennema-Nio <postgres@jeltef.nl> wrote:
>
> I feel like this is the type of change where there's not much
> discussion to be had. And the only way to resolve it is to use some
> voting to gauge community opinion.
>
> So my suggestion is for people to respond with -1, -0.5, +-0, +0.5, or
> +1 to indicate support against/for the change.
>
> I'll start: +1
>
> Attached is a patch that removes -Wdeclaration-after-statement in the
> codebase. This is mainly to be able to add it to the commitfest, to
> hopefully get a decent amount of responses.


I am new to c.
gcc has many options (https://man7.org/linux/man-pages/man1/gcc.1.html)
IIUC, postgres have some required CFLAGS,
While doing the test (http://cfbot.cputube.org/next.html), we also
have some CFLAGS
I am not sure they are the same.

if we can list these CFLAGS information in the
https://wiki.postgresql.org/wiki/So,_you_want_to_be_a_developer%3F
would be helpful.

But not voting because I am not sure of the implication.



pgsql-hackers by date:

Previous
From: Corey Huinker
Date:
Subject: Re: Document efficient self-joins / UPDATE LIMIT techniques.
Next
From: Michael Paquier
Date:
Subject: Re: Checking MINIMUM_VERSION_FOR_WAL_SUMMARIES