Re: pgsql: Clean up warnings from -Wimplicit-fallthrough. - Mailing list pgsql-hackers

From Tom Lane
Subject Re: pgsql: Clean up warnings from -Wimplicit-fallthrough.
Date
Msg-id 7773.1525232005@sss.pgh.pa.us
Whole thread Raw
In response to Re: pgsql: Clean up warnings from -Wimplicit-fallthrough.  (Andres Freund <andres@anarazel.de>)
Responses Re: pgsql: Clean up warnings from -Wimplicit-fallthrough.  (Peter Eisentraut <peter.eisentraut@2ndquadrant.com>)
List pgsql-hackers
Andres Freund <andres@anarazel.de> writes:
> On 2018-05-01 23:35:18 +0000, Tom Lane wrote:
>> Clean up warnings from -Wimplicit-fallthrough.

> I found one more oddity with the current committed state: ...
> It seems that gcc gets confused by the #ifdef ECONNRESET.

Yeah, there's a gcc bug or three about that:

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77817

> ... But the nicer fix seems to be to move the
> FALL THRU into the ifdef, which even seems more correct when nitpicking
> in god mode.

I thought about that, and didn't like it much.  I hoped maybe the gcc guys
would fix it soon, but after reading their bugzilla entry more closely,
it sounds like we shouldn't hold our breath.  So if you want to do it
like that, OK by me.

            regards, tom lane


pgsql-hackers by date:

Previous
From: Andres Freund
Date:
Subject: Re: A few warnings on Windows
Next
From: Tom Lane
Date:
Subject: Re: A few warnings on Windows