Re: GCC warning in back branches - Mailing list pgsql-hackers

From Tom Lane
Subject Re: GCC warning in back branches
Date
Msg-id 2151315.1613353285@sss.pgh.pa.us
Whole thread Raw
In response to Re: GCC warning in back branches  (Michael Paquier <michael@paquier.xyz>)
List pgsql-hackers
Michael Paquier <michael@paquier.xyz> writes:
> On Mon, Feb 15, 2021 at 02:15:51PM +1300, Thomas Munro wrote:
>> I propose the attached.

> We usually don't bother much about compilation warnings in stable
> branches as long as they are not real bugs, and these are the oldest
> stable ones.  So why here?  I would have patched the top of the
> function if it were me, btw.

If somebody were running a buildfarm member with recent gcc
and -Werror, we'd pretty much have to fix it.

I'd say the real policy is that we don't worry about
uninitialized-variable warnings from old compiler versions,
on the theory that they're probably compiler shortcomings.
But I'd be inclined to fix anything from a current gcc version.

            regards, tom lane



pgsql-hackers by date:

Previous
From: Michael Paquier
Date:
Subject: Re: GCC warning in back branches
Next
From: Thomas Munro
Date:
Subject: Re: Use pg_pwrite() in pg_test_fsync