From f80a0e7bc0095a4f87c59422c13904fb6f3ab14a Mon Sep 17 00:00:00 2001 From: Alastair Hughes Date: Mon, 8 May 2017 07:49:17 +1200 Subject: [PATCH] Remove extra braces to fix build with gcc 7 GCC 7.1 warns on "implicit fallthrough" in case statements. To avoid false positives it will attempt to match comments directly before the fall through in question, and if any match some form of "fall through" the fall through is ignored. For this particular instance extra braces prevent the comment being recognized, as the following line is not a label or a new case; remove the excess braces to let GCC recognize the comments. Signed-off-by: Alastair Hughes --- src/parse/parse.c | 4 ---- 1 file changed, 4 deletions(-) diff --git a/src/parse/parse.c b/src/parse/parse.c index b65b0f5..7084a9e 100644 --- a/src/parse/parse.c +++ b/src/parse/parse.c @@ -2091,7 +2091,6 @@ css_error parseMalformedDeclaration(css_parser *parser) switch (state->substate) { case Initial: - { /* Clear stack of open items */ while (parserutils_stack_pop(parser->open_items, NULL) == PARSERUTILS_OK) @@ -2099,7 +2098,6 @@ css_error parseMalformedDeclaration(css_parser *parser) state->substate = Go; /* Fall through */ - } case Go: while (1) { char want; @@ -2289,7 +2287,6 @@ css_error parseMalformedAtRule(css_parser *parser) switch (state->substate) { case Initial: - { /* Clear stack of open items */ while (parserutils_stack_pop(parser->open_items, NULL) == PARSERUTILS_OK) @@ -2297,7 +2294,6 @@ css_error parseMalformedAtRule(css_parser *parser) state->substate = Go; /* Fall through */ - } case Go: while (1) { char want; -- 2.12.2