diff options
author | D G Turner | 2019-10-15 21:46:14 +0100 |
---|---|---|
committer | D G Turner | 2019-10-15 21:46:14 +0100 |
commit | 66011fcae80b84620d2b1f0c466f75e03fb113d7 (patch) | |
tree | 0566a5cf1d7a5e401e5fa022f0b295e88f8e1f28 /engines/avalanche/dialogs.cpp | |
parent | 06b038a1c15d1486fc3466050391e12d22b5356a (diff) | |
download | scummvm-rg350-66011fcae80b84620d2b1f0c466f75e03fb113d7.tar.gz scummvm-rg350-66011fcae80b84620d2b1f0c466f75e03fb113d7.tar.bz2 scummvm-rg350-66011fcae80b84620d2b1f0c466f75e03fb113d7.zip |
AVALANCHE: Fix Missing Default Switch Cases
These are flagged by GCC if -Wswitch-default is enabled.
Diffstat (limited to 'engines/avalanche/dialogs.cpp')
-rw-r--r-- | engines/avalanche/dialogs.cpp | 13 |
1 files changed, 12 insertions, 1 deletions
diff --git a/engines/avalanche/dialogs.cpp b/engines/avalanche/dialogs.cpp index d1a7234b03..3ccba4e139 100644 --- a/engines/avalanche/dialogs.cpp +++ b/engines/avalanche/dialogs.cpp @@ -97,6 +97,7 @@ void Dialogs::setReadyLight(byte state) { // TODO: Implement different patterns for green color. Color color = kColorBlack; switch (state) { + default: case 0: color = kColorBlack; break; // Off @@ -459,6 +460,7 @@ void Dialogs::drawScroll(DialogFunctionType modeFunc) { byte iconIndent = 0; switch (_useIcon) { + default: case 0: iconIndent = 0; break; // No icon. @@ -495,6 +497,8 @@ void Dialogs::drawScroll(DialogFunctionType modeFunc) { _vm->_graphics->drawShadowBox(_shadowBoxX - 65, _shadowBoxY - 24, _shadowBoxX - 5, _shadowBoxY - 10, "Yes."); _vm->_graphics->drawShadowBox(_shadowBoxX + 5, _shadowBoxY - 24, _shadowBoxX + 65, _shadowBoxY - 10, "No."); break; + default: + break; } if (center) @@ -734,7 +738,7 @@ void Dialogs::displayText(Common::String text) { return; break; - // CHECME: The whole kControlNegative block seems completely unused, as the only use (the easter egg check) is a false positive + // CHECKME: The whole kControlNegative block seems completely unused, as the only use (the easter egg check) is a false positive case kControlNegative: switch (_param) { case 1: @@ -783,6 +787,8 @@ void Dialogs::displayText(Common::String text) { displayText(_vm->getItem(j) + ", " + kControlToBuffer); } break; + default: + break; } break; case kControlIcon: @@ -1031,6 +1037,8 @@ void Dialogs::talkTo(byte whom) { case 3: displayScrollChain('Q', 30); // Need any help with the game? return; + default: + break; } } else { displayScrollChain('Q', 42); // Haven't talked to Crapulus. Go and talk to him. @@ -1085,6 +1093,9 @@ void Dialogs::talkTo(byte whom) { } } break; + + default: + break; } // On a subject. Is there any reason to block it? } else if ((whom == kPeopleAyles) && (!_vm->_aylesIsAwake)) { |