[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] fallthrough: update for GCC 7/8
From: |
Jim Meyering |
Subject: |
Re: [PATCH] fallthrough: update for GCC 7/8 |
Date: |
Mon, 22 May 2017 08:06:40 -0700 |
On Mon, May 22, 2017 at 4:50 AM, Pádraig Brady <address@hidden> wrote:
> On 22/05/17 07:07, Bernhard Voelker wrote:
>> On 05/20/2017 02:12 AM, Jim Meyering wrote:
>>> * lib/quotearg.c (FALLTHROUGH): New macro.
>>> Use it whenever one switch case falls through into the next,
>>> replacing "/* Fall through */" comments. This exposed one
>>> instance of an unwarranted "fall through" comment: unwarranted
>>> because it preceded a "goto" label not a case statement.
>>
>> Here's a diff output with one more line of context:
>>
>> @@ -504,9 +512,8 @@ quotearg_buffer_restyled (char *buffer, size_t
>> buffersize,
>> c_and_shell_escape:
>> if (quoting_style == shell_always_quoting_style
>> && elide_outer_quotes)
>> goto force_outer_quoting_style;
>> - /* Fall through. */
>> c_escape:
>> if (backslash_escapes)
>> {
>> c = esc;
>>
>> IMO the "fall through" comment was warranted ... in the else case.
>
> Agreed. Pushed that change in your name.
> Thanks to both of you for the cleanups.
Indeed. Thanks.