dfa.c evokes a new warning from GCC 7. I fixed it with this:
From 5f919be9d0db97e8daecaa61e6f4ff3b96b2a0fe Mon Sep 17 00:00:00 2001
From: Jim Meyering <meyer...@fb.com>
Date: Wed, 28 Sep 2016 21:20:38 -0700
Subject: [PATCH] dfa: build: avoid warning from GCC 7's new
 -Wimplicit-fallthrough

* lib/dfa.c (dfassbuild): Mark the end of this case with a
/* fallthrough */ comment.
---
 ChangeLog | 4 ++++
 lib/dfa.c | 1 +
 2 files changed, 5 insertions(+)

diff --git a/ChangeLog b/ChangeLog
index 21a55e8..06174b6 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,9 @@
 2016-09-28  Jim Meyering  <meyer...@fb.com>

+       dfa: build: avoid warning from GCC 7's new -Wimplicit-fallthrough
+       * lib/dfa.c (dfassbuild): Mark the end of this case with a
+       /* fallthrough */ comment.
+
        getprogname: avoid __progname vs program_invocation_short_name pitfall
        I.e., don't let the OpenBSD 5.1 fix induce failure when using newer
        glibc.  Would have caused failure with Fedora 25's glibc-2.24-3, but
diff --git a/lib/dfa.c b/lib/dfa.c
index 4c41fa6..c4dc626 100644
--- a/lib/dfa.c
+++ b/lib/dfa.c
@@ -3434,6 +3434,7 @@ dfassbuild (struct dfa *d)
               sup->tokens[j++] = EMPTY;
               break;
             }
+          /* fallthrough */
         default:
           sup->tokens[j++] = d->tokens[i];
           if ((0 <= d->tokens[i] && d->tokens[i] < NOTCHAR)
-- 
2.7.4

Reply via email to