From aee09409e8b30dbd8ea10c9190b85037fe8458c9 Mon Sep 17 00:00:00 2001 From: D G Turner Date: Sat, 30 Nov 2019 22:08:44 +0000 Subject: IMAGE: Fix Missing Default Switch Cases These are flagged by GCC if -Wswitch-default is enabled. --- image/codecs/bmp_raw.cpp | 2 ++ image/codecs/indeo/indeo_dsp.cpp | 2 ++ image/codecs/indeo4.cpp | 2 ++ image/codecs/indeo5.cpp | 5 +++++ image/codecs/smc.cpp | 3 +++ image/codecs/svq1.cpp | 6 ++++++ 6 files changed, 20 insertions(+) (limited to 'image/codecs') diff --git a/image/codecs/bmp_raw.cpp b/image/codecs/bmp_raw.cpp index 149136c5b4..257fa02b82 100644 --- a/image/codecs/bmp_raw.cpp +++ b/image/codecs/bmp_raw.cpp @@ -140,6 +140,8 @@ Graphics::PixelFormat BitmapRawDecoder::getPixelFormat() const { case 24: case 32: return Graphics::PixelFormat(4, 8, 8, 8, 8, 8, 16, 24, 0); + default: + break; } error("Unhandled BMP raw %dbpp", _bitsPerPixel); diff --git a/image/codecs/indeo/indeo_dsp.cpp b/image/codecs/indeo/indeo_dsp.cpp index 95ce3f3bf5..8e281aa530 100644 --- a/image/codecs/indeo/indeo_dsp.cpp +++ b/image/codecs/indeo/indeo_dsp.cpp @@ -563,6 +563,8 @@ static void iviMc ## size ##x## size ## suffix(int16 *buf, \ for (int j = 0; j < size; j++) \ OP(buf[j], (refBuf[j] + refBuf[j+1] + wptr[j] + wptr[j+1]) >> 2); \ break; \ + default: \ + break; \ } \ } \ \ diff --git a/image/codecs/indeo4.cpp b/image/codecs/indeo4.cpp index a4eba85f49..8385658e3c 100644 --- a/image/codecs/indeo4.cpp +++ b/image/codecs/indeo4.cpp @@ -251,6 +251,8 @@ void Indeo4Decoder::switchBuffers() { case IVI4_FRAMETYPE_INTER: isPrevRef = 1; break; + default: + break; } switch (_ctx._frameType) { diff --git a/image/codecs/indeo5.cpp b/image/codecs/indeo5.cpp index 790bdec87a..e36fb55d97 100644 --- a/image/codecs/indeo5.cpp +++ b/image/codecs/indeo5.cpp @@ -176,6 +176,7 @@ void Indeo5Decoder::switchBuffers() { break; case FRAMETYPE_INTER_NOREF: + default: break; } @@ -192,6 +193,7 @@ void Indeo5Decoder::switchBuffers() { case FRAMETYPE_INTER_SCAL: case FRAMETYPE_INTER_NOREF: case FRAMETYPE_NULL: + default: break; } } @@ -513,6 +515,9 @@ int Indeo5Decoder::decode_gop_header() { band->_scan = _ffIviDirectScan4x4; band->_transformSize = 4; break; + + default: + break; } band->_is2dTrans = band->_invTransform == IndeoDSP::ffIviInverseSlant8x8 || diff --git a/image/codecs/smc.cpp b/image/codecs/smc.cpp index 54493d0c34..716b2f96fd 100644 --- a/image/codecs/smc.cpp +++ b/image/codecs/smc.cpp @@ -380,6 +380,9 @@ const Graphics::Surface *SMCDecoder::decodeFrame(Common::SeekableReadStream &str case 0xF0: warning("0xF0 opcode seen in SMC chunk (contact the developers)"); break; + + default: + break; } } diff --git a/image/codecs/svq1.cpp b/image/codecs/svq1.cpp index eba5fb872a..5481967c07 100644 --- a/image/codecs/svq1.cpp +++ b/image/codecs/svq1.cpp @@ -657,6 +657,8 @@ bool SVQ1Decoder::svq1MotionInterBlock(Common::BitStream32BEMSB *ss, byte *curre case 3: putPixels16XY2C(dst, src, pitch, 16); break; + default: + break; } return true; @@ -737,6 +739,8 @@ bool SVQ1Decoder::svq1MotionInter4vBlock(Common::BitStream32BEMSB *ss, byte *cur case 3: putPixels8XY2C(dst, src, pitch, 8); break; + default: + break; } // select next block @@ -789,6 +793,8 @@ bool SVQ1Decoder::svq1DecodeDeltaBlock(Common::BitStream32BEMSB *ss, byte *curre case SVQ1_BLOCK_INTRA: resultValid = svq1DecodeBlockIntra(ss, current, pitch); break; + default: + break; } return resultValid; -- cgit v1.2.3