Re: pgsql: aio: Add core asynchronous I/O infrastructure

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

In response to

Browse pgsql-committers by date

  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