Incorrect declaration in the switch/case (break before the expression)

RESOLVED FIXED in Firefox 56

Status

()

enhancement
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: sylvestre, Assigned: sylvestre)

Tracking

(Blocks 1 bug, {coverity})

unspecified
mozilla56
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox56 fixed)

Details

(Whiteboard: [CID 1415663])

Attachments

(1 attachment)

Thanks, yes, this needs a fix!
Comment on attachment 8889339 [details]
Bug 1383657 - Fix the incorrectly placed 'break;' in the switch/case declaration =mayhemer

https://reviewboard.mozilla.org/r/160418/#review165682
Attachment #8889339 - Flags: review+
Keywords: checkin-needed
(using autoland in mozreview)
Pushed by sledru@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/54deee11a7c5
Fix the incorrectly placed 'break;' in the switch/case declaration r=mayhemer=mayhemer
https://hg.mozilla.org/mozilla-central/rev/54deee11a7c5
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla56
This bug would also have been caught by clang's -Wunreachable-code-break warning ... if it had been enabled. :) I have local patches to enable these warnings. I should post them for review soon.
yeah! we should do that
fyi, gcc 7.1 has -Wswitch-unreachable which found bug 1383007
You need to log in before you can comment on or make changes to this bug.