Re: Warning-suppression fixes we ought to back-patch - Mailing list pgsql-hackers

From Peter Eisentraut
Subject Re: Warning-suppression fixes we ought to back-patch
Date
Msg-id a159cee6-5b19-44c3-8457-45b25345412b@eisentraut.org
Whole thread Raw
In response to Warning-suppression fixes we ought to back-patch  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: Warning-suppression fixes we ought to back-patch
List pgsql-hackers
On 24.02.26 17:24, Tom Lane wrote:
> I looked through the buildfarm's compiler warnings on back branches,
> which is something I rarely do, and noted a bunch of warnings that
> we are not seeing on master.  That's because of recent commits that
> aimed to silence those warning types.  Most of the noise could be
> suppressed by back-patching these two changes:
> 
> 0909380e4    Allow PG_PRINTF_ATTRIBUTE to be different in C and C++ code.
> 
> This only matters for builds that are combining gcc with clang++ or
> vice versa, but we have a dozen or so BF animals that are like that,
> and each one is spewing a hundred "unrecognized format function type"
> warnings.
> 
> 8f1791c61    Fix some cases of indirectly casting away const.

No problem backpatching these.

> While we've done a lot of const-cleanliness work recently, this patch
> should be enough to silence most of the cast-away-const warnings I'm
> seeing in the BF back branches.  Only "midge" is showing them at the
> moment, but more people will be seeing them as gcc 16 gets more
> widespread.  (I'd probably skip the ecpg bits though, as they are
> a bit more convoluted than the rest, so the risk/benefit ratio
> seems poor.)

These warnings came in from the new glibc version 2.43, not from the 
compiler.




pgsql-hackers by date:

Previous
From: Yura Sokolov
Date:
Subject: Re: Fix bug in multixact Oldest*MXactId initialization and access
Next
From: Tom Lane
Date:
Subject: Re: New isolation test insert-conflict-do-update-4 outputs rows in alternative ordering