Re: pgsql: aio: Add core asynchronous I/O infrastructure - Mailing list pgsql-committers

From Tom Lane
Subject Re: pgsql: aio: Add core asynchronous I/O infrastructure
Date
Msg-id 713810.1742402548@sss.pgh.pa.us
Whole thread Raw
In response to Re: pgsql: aio: Add core asynchronous I/O infrastructure  (Andres Freund <andres@anarazel.de>)
List pgsql-committers
Andres Freund <andres@anarazel.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
initializationof 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



pgsql-committers by date:

Previous
From: Andres Freund
Date:
Subject: Re: pgsql: aio: Add core asynchronous I/O infrastructure
Next
From: Amit Kapila
Date:
Subject: pgsql: pg_createsubscriber: Add -R publications option.