From 70a2ca8b7d64e993fdab89c7b8b0d30ab4334824 Mon Sep 17 00:00:00 2001 From: Torbjörn Andersson Date: Sun, 6 Aug 2017 13:30:51 +0200 Subject: JANITORIAL: Silence more GCC 7 warnings All these fall through were marked as deliberate, so again I've only changed the comment to silence GCC. --- engines/sci/graphics/plane32.cpp | 3 ++- engines/sci/graphics/text16.cpp | 4 ++-- 2 files changed, 4 insertions(+), 3 deletions(-) (limited to 'engines/sci/graphics') diff --git a/engines/sci/graphics/plane32.cpp b/engines/sci/graphics/plane32.cpp index 50a34656de..d9e0d2d9ff 100644 --- a/engines/sci/graphics/plane32.cpp +++ b/engines/sci/graphics/plane32.cpp @@ -766,7 +766,8 @@ void Plane::setType() { _type = kPlaneTypeTransparentPicture; break; } - // fall through for games without transparent picture planes + // The game doesn't have transparent picture planes + // fall through default: if (!g_sci->_features->hasTransparentPicturePlanes() || _type != kPlaneTypeTransparentPicture) { _type = kPlaneTypePicture; diff --git a/engines/sci/graphics/text16.cpp b/engines/sci/graphics/text16.cpp index 903136c29d..9808180716 100644 --- a/engines/sci/graphics/text16.cpp +++ b/engines/sci/graphics/text16.cpp @@ -225,7 +225,7 @@ int16 GfxText16::GetLongest(const char *&textPtr, int16 maxWidth, GuiResourceId if ((*(const byte *)(textPtr + 1)) == 0xA) { curCharCount++; textPtr++; } - // it's meant to pass through here + // fall through case 0xA: case 0x9781: // this one is used by SQ4/japanese as line break as well (was added for SCI1/PC98) curCharCount++; textPtr++; @@ -233,7 +233,7 @@ int16 GfxText16::GetLongest(const char *&textPtr, int16 maxWidth, GuiResourceId // skip another byte in case char is double-byte (PC-98) curCharCount++; textPtr++; } - // and it's also meant to pass through here + // fall through case 0: SetFont(previousFontId); _ports->penColor(previousPenColor); -- cgit v1.2.3