From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Andres Freund <andres(at)anarazel(dot)de> |
Cc: | pgsql-committers(at)lists(dot)postgresql(dot)org |
Subject: | Re: pgsql: aio: Add core asynchronous I/O infrastructure |
Date: | 2025-03-19 16:42:28 |
Message-ID: | 713810.1742402548@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
Andres Freund <andres(at)anarazel(dot)de> writes:
> On 2025-03-18 11:26:15 -0400, Tom Lane wrote:
>> Well, if we don't want to fix it I can just adjust my warning-scraping
>> script to ignore these. Let's wait a bit and see if any newer
>> compilers warn similarly before deciding.
> FWIW, adding a few casts does silence the warnings, at least on clang-3.9,
> which I had around.
As of now, the only BF members showing these warnings are ayu (clang
version 4.0.1-10~deb9u2), batfish (clang version 5.0.0-3~16.04.1),
and demoiselle (clang version 5.0.1). None of those seem like
versions that anyone would use for code development anymore (clang 6.0
was released in March 2018), so I'm not feeling like these warnings
are important to silence. Let's just ignore them.
> But there also are other warnings on that compiler, mainly stuff like:
> ../../home/andres/src/postgresql/src/interfaces/libpq/fe-auth-oauth-curl.c:1437:28: warning: suggest braces around initialization of subobject [-Wmissing-braces]
> struct itimerspec spec = {0};
And even more so that, if it's so old it's not even in the buildfarm
anymore. I do care about -Wmissing-braces on a version developers
might use, but ...
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | Amit Kapila | 2025-03-20 06:59:31 | pgsql: pg_createsubscriber: Add -R publications option. |
Previous Message | Andres Freund | 2025-03-19 14:29:19 | Re: pgsql: aio: Add core asynchronous I/O infrastructure |