Skip to content

Commit b29158d

Browse files
Jeremy Daykripken
authored andcommitted
remove implicit fallthroughs (#1194) (#1196)
1 parent 0532093 commit b29158d

3 files changed

Lines changed: 599 additions & 703 deletions

File tree

src/tools/translate-to-fuzz.h

Lines changed: 11 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -639,9 +639,17 @@ class TranslateToFuzzReader {
639639
}
640640
}
641641
switch (conditions) {
642-
case 0: if (!oneIn(4)) continue;
643-
case 1: if (!oneIn(2)) continue;
644-
default: if (oneIn(conditions + 1)) continue;
642+
case 0: {
643+
if (!oneIn(4)) continue;
644+
break;
645+
}
646+
case 1: {
647+
if (!oneIn(2)) continue;
648+
break;
649+
}
650+
default: {
651+
if (oneIn(conditions + 1)) continue;
652+
}
645653
}
646654
return builder.makeBreak(name);
647655
}

src/wasm/wasm-s-parser.cpp

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -874,6 +874,7 @@ Expression* SExpressionWasmBuilder::makeExpression(Element& s) {
874874
}
875875
case 'w': {
876876
if (!strncmp(str, "wake", strlen("wake"))) return makeAtomicWake(s);
877+
abort_on(str);
877878
}
878879
default: abort_on(str);
879880
}

0 commit comments

Comments
 (0)