Skip to content
This repository has been archived by the owner on Oct 12, 2022. It is now read-only.

Commit

Permalink
Fix bug and build breakage caused by implicit case fallthrough
Browse files Browse the repository at this point in the history
  • Loading branch information
CyberShadow committed Sep 3, 2011
1 parent 6f4bf93 commit 7fb7227
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions src/core/demangle.d
Original file line number Diff line number Diff line change
Expand Up @@ -1240,6 +1240,7 @@ private struct Demangle
// \", \?
case '\\':
put( "'\\\\'" );
return;
case '\a':
put( "'\\a'" );
return;
Expand All @@ -1248,6 +1249,7 @@ private struct Demangle
return;
case '\f':
put( "'\\f'" );
return;
case '\n':
put( "'\\n'" );
return;
Expand Down

3 comments on commit 7fb7227

@WalterBright
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

still getting:

src\core\demangle.d(1067): Warning: statement is not reachable
src\core\demangle.d(1067): Warning: statement is not reachable

@CyberShadow
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, didn't realize warnings are treated as errors in druntime. Pushed another commit with what I think is the correct fix.

@CyberShadow
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like it's not showing up. Just replace the statement on that line with assert(0);. Sorry about the mess.

Please sign in to comment.