| From: | Andres Freund <andres(at)anarazel(dot)de> |
|---|---|
| To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
| Cc: | Peter Eisentraut <peter(dot)eisentraut(at)enterprisedb(dot)com>, pgsql-hackers(at)postgresql(dot)org, David Rowley <dgrowleyml(at)gmail(dot)com>, Dagfinn Ilmari Mannsåker <ilmari(at)ilmari(dot)org> |
| Subject: | Re: perl 5.36, C99, -Wdeclaration-after-statement -Wshadow=compatible-local |
| Date: | 2022-11-03 00:03:34 |
| Message-ID: | 20221103000334.sbh5jqw37i3y7pc3@awork3.anarazel.de |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Hi,
On 2022-11-02 19:57:45 -0400, Tom Lane wrote:
> Andres Freund <andres(at)anarazel(dot)de> writes:
> > On 2022-11-01 17:00:27 -0400, Peter Eisentraut wrote:
> >> Python has the same issues. There are a few other Python-embedding projects
> >> that use -Wdeclaration-after-statement and complain if the Python headers
> >> violate it. But it's getting tedious. -isystem would be a better solution.
>
> > Which dependencies should we convert to -isystem?
>
> Color me confused about what's being discussed here. I see nothing
> in the gcc manual suggesting that -isystem has any effect on warning
> levels?
It's only indirectly explained :(
The -isystem and -idirafter options also mark the directory as a system directory, so that it gets the same special treatment that is applied to
the standard system directories.
and then https://gcc.gnu.org/onlinedocs/cpp/System-Headers.html
Greetings,
Andres Freund
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Thomas Munro | 2022-11-03 01:10:22 | Re: Improve logging when using Huge Pages |
| Previous Message | Tom Lane | 2022-11-02 23:57:45 | Re: perl 5.36, C99, -Wdeclaration-after-statement -Wshadow=compatible-local |