From: | Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Andres Freund <andres(at)anarazel(dot)de> |
Cc: | pgsql-hackers(at)lists(dot)postgresql(dot)org |
Subject: | Re: pgsql: Clean up warnings from -Wimplicit-fallthrough. |
Date: | 2018-05-03 13:36:17 |
Message-ID: | 9175dfb6-c981-6075-6cb7-bc1cc32c092f@2ndquadrant.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers |
On 5/1/18 23:33, Tom Lane wrote:
> Andres Freund <andres(at)anarazel(dot)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
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2018-05-03 15:33:02 | pgsql: Further improve code for probing the availability of ARM CRC ins |
Previous Message | Peter Eisentraut | 2018-05-03 12:43:07 | pgsql: Fix SPI error cleanup and memory leak |
From | Date | Subject | |
---|---|---|---|
Next Message | Pavlo Golub | 2018-05-03 14:05:12 | Re: Issues while building PG in MS Windows, using MSYS2 and MinGW-w64 |
Previous Message | Craig Ringer | 2018-05-03 12:55:01 | Re: Anyone keep mirrors of old packages from apt.postgresql.org? |