Re: GCC warning in back branches
Tom Lane <tgl@sss.pgh.pa.us>
From: Tom Lane <tgl@sss.pgh.pa.us>
To: Michael Paquier <michael@paquier.xyz>
Cc: Thomas Munro <thomas.munro@gmail.com>,
pgsql-hackers <pgsql-hackers@postgresql.org>
Date: 2021-02-15T01:41:25Z
Lists: 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
Commits
-
Fix compiler warning in back branches (9.6, 10).
- 618d139f82d4 9.6.22 landed
- 26812bcaa664 10.17 landed
-
Fix assorted compiler warnings seen in the buildfarm.
- fbb2e9a030ee 11.0 cited