diff options
author | D G Turner | 2018-11-13 12:10:30 +0000 |
---|---|---|
committer | D G Turner | 2018-11-13 12:10:30 +0000 |
commit | 09abcc5c2c59341fadd0450db3755e1d2662655d (patch) | |
tree | eddfbc516069531d83a92f0b7ce6f4b72837610b /engines/sci/graphics | |
parent | c006f405f3b9e7109ab5f0dd08671d26c582f886 (diff) | |
download | scummvm-rg350-09abcc5c2c59341fadd0450db3755e1d2662655d.tar.gz scummvm-rg350-09abcc5c2c59341fadd0450db3755e1d2662655d.tar.bz2 scummvm-rg350-09abcc5c2c59341fadd0450db3755e1d2662655d.zip |
SCI: Fix Fall Through Compiler Warnings.
Diffstat (limited to 'engines/sci/graphics')
-rw-r--r-- | engines/sci/graphics/text16.cpp | 4 | ||||
-rw-r--r-- | engines/sci/graphics/view.cpp | 2 |
2 files changed, 6 insertions, 0 deletions
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)); |