From 09abcc5c2c59341fadd0450db3755e1d2662655d Mon Sep 17 00:00:00 2001
From: D G Turner
Date: Tue, 13 Nov 2018 12:10:30 +0000
Subject: SCI: Fix Fall Through Compiler Warnings.

---
 engines/sci/engine/ksound.cpp      | 2 ++
 engines/sci/engine/scriptdebug.cpp | 2 ++
 engines/sci/graphics/text16.cpp    | 4 ++++
 engines/sci/graphics/view.cpp      | 2 ++
 4 files changed, 10 insertions(+)

(limited to 'engines/sci')

diff --git a/engines/sci/engine/ksound.cpp b/engines/sci/engine/ksound.cpp
index 4d171bdd1c..e6cd4ffc7f 100644
--- a/engines/sci/engine/ksound.cpp
+++ b/engines/sci/engine/ksound.cpp
@@ -257,6 +257,8 @@ reg_t kDoAudio(EngineState *s, int argc, reg_t *argv) {
 			debugC(kDebugLevelSound, "kDoAudio: CD audio subop");
 			return kDoCdAudio(s, argc - 1, argv + 1);
 		}
+		// fall through
+		// FIXME: fall through intended?
 
 		// 3 new subops in Pharkas CD (including CD demo). kDoAudio in Pharkas sits at seg026:038C
 	case 11:
diff --git a/engines/sci/engine/scriptdebug.cpp b/engines/sci/engine/scriptdebug.cpp
index f884e4da60..f5c1f8de8f 100644
--- a/engines/sci/engine/scriptdebug.cpp
+++ b/engines/sci/engine/scriptdebug.cpp
@@ -450,6 +450,8 @@ void SciEngine::scriptDebug() {
 				case kDebugSeekSpecialCallk:
 					if (paramb1 != _debugState.seekSpecial)
 						return;
+					// fall through
+					// FIXME: fall through intended?
 
 				case kDebugSeekCallk:
 					if (op != op_callk)
diff --git a/engines/sci/graphics/text16.cpp b/engines/sci/graphics/text16.cpp
index 9808180716..a661de63f9 100644
--- a/engines/sci/graphics/text16.cpp
+++ b/engines/sci/graphics/text16.cpp
@@ -367,6 +367,8 @@ void GfxText16::Width(const char *text, int16 from, int16 len, GuiResourceId org
 					len -= CodeProcessing(text, orgFontId, 0, false);
 					break;
 				}
+				// fall through
+				// FIXME: fall through intended?
 			default:
 				textHeight = MAX<int16> (textHeight, _ports->_curPort->fontHeight);
 				textWidth += _font->getCharWidth(curChar);
@@ -471,6 +473,8 @@ void GfxText16::Draw(const char *text, int16 from, int16 len, GuiResourceId orgF
 				len -= CodeProcessing(text, orgFontId, orgPenColor, true);
 				break;
 			}
+			// fall through
+			// FIXME: fall through intended?
 		default:
 			charWidth = _font->getCharWidth(curChar);
 			// clear char
diff --git a/engines/sci/graphics/view.cpp b/engines/sci/graphics/view.cpp
index 02ffc240d9..319e3ac4fb 100644
--- a/engines/sci/graphics/view.cpp
+++ b/engines/sci/graphics/view.cpp
@@ -140,6 +140,7 @@ void GfxView::initData(GuiResourceId resourceId) {
 	switch (curViewType) {
 	case kViewEga: // SCI0 (and Amiga 16 colors)
 		isEGA = true;
+		// fall through
 	case kViewAmiga: // Amiga ECS (32 colors)
 	case kViewAmiga64: // Amiga AGA (64 colors)
 	case kViewVga: // View-format SCI1
@@ -578,6 +579,7 @@ void unpackCelData(const SciSpan<const byte> &inBuffer, SciSpan<byte> &celBitmap
 			switch (curByte & 0xC0) {
 			case 0x40: // copy bytes as is (In copy case, runLength can go up to 127 i.e. pixel & 0x40). Fixes bug #3135872.
 				runLength += 64;
+				// fall through
 			case 0x00: // copy bytes as-is
 				if (!literalPos) {
 					memcpy(outPtr + pixelNr,        rlePtr, MIN<uint16>(runLength, pixelCount - pixelNr));
-- 
cgit v1.2.3