Re: Warning-suppression fixes we ought to back-patch

From: Peter Eisentraut <peter(at)eisentraut(dot)org>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, pgsql-hackers(at)lists(dot)postgresql(dot)org
Subject: Re: Warning-suppression fixes we ought to back-patch
Date: 2026-02-25 15:43:07
Message-ID: a159cee6-5b19-44c3-8457-45b25345412b@eisentraut.org
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: 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.

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2026-02-25 15:52:22 Re: New isolation test insert-conflict-do-update-4 outputs rows in alternative ordering
Previous Message Yura Sokolov 2026-02-25 15:16:27 Re: Fix bug in multixact Oldest*MXactId initialization and access