| 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.
| 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 |