On 5/1/18 23:33, Tom Lane wrote:
> 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.
It sounds like we have a handle on all the potential issues for now, so
I think it would be OK to add this warning to the standard set on master.
--
Peter Eisentraut http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
