Fix enum switch/case statements with unhandled values.
authorCarl Hetherington <cth@carlh.net>
Mon, 11 Jan 2016 10:43:43 +0000 (10:43 +0000)
committerCarl Hetherington <cth@carlh.net>
Tue, 3 Dec 2019 15:44:08 +0000 (16:44 +0100)
src/AS_DCP_MPEG2.cpp
src/AS_DCP_PCM.cpp
src/JP2K_Codestream_Parser.cpp
src/KM_log.cpp
src/MPEG2_Parser.cpp
src/WavFileWriter.h
src/kmfilegen.cpp

index b7877b54549be8a135afcd77432619bdc55ae2f8..598e585d303607f5095671c31c2d790aaa8038a7 100755 (executable)
@@ -607,6 +607,7 @@ ASDCP::MPEG2::MXFWriter::h__Writer::WriteFrame(const FrameBuffer& FrameBuf, AESE
     case FRAME_I: Flags = 0x00; break;
     case FRAME_P: Flags = 0x22; break;
     case FRAME_B: Flags = 0x33; break;
+    default: break;
     }
 
   if ( FrameBuf.GOPStart() )
index d90581440474498fe2fdf7ee15f375b573e88f98..1ca197c4c8a21b916c4b210a90ea14261676ffb4 100755 (executable)
@@ -81,6 +81,9 @@ ASDCP::PCM_ADesc_to_MD(PCM::AudioDescriptor& ADesc, MXF::WaveAudioDescriptor* AD
       case PCM::CF_CFG_6:
        ADescObj->ChannelAssignment = DefaultSMPTEDict().Type(MDD_DCAudioChannelCfg_MCA).ul;
        break;
+
+      default:
+       break;
     }
 
   return RESULT_OK;
index 0ed26492e82e4f8ccce9295d92c3247a219108e0..5401db2e2d5f807f1a1f8cf9ecd2ec044fa47429 100755 (executable)
@@ -178,6 +178,9 @@ ASDCP::JP2K::ParseMetadataIntoDesc(const FrameBuffer& FB, PictureDescriptor& PDe
          memcpy(&PDesc.QuantizationDefault, NextMarker.m_Data, NextMarker.m_DataSize);
          PDesc.QuantizationDefault.SPqcdLength = NextMarker.m_DataSize - 1;
          break;
+
+       default:
+         break;
        }
     }
 
index 182d91e6ee2cb2730230cc39672e0d5ad8fbd562..885dd1262a9ac6755dbab1b94ae0b303dde790a3 100755 (executable)
@@ -202,6 +202,7 @@ Kumu::SyslogLogSink::WriteEntry(const LogEntry& Entry)
     case Kumu::LOG_NOTICE:  priority = SYSLOG_NOTICE; break;
     case Kumu::LOG_INFO:    priority = SYSLOG_INFO; break;
     case Kumu::LOG_DEBUG:   priority = SYSLOG_DEBUG; break;
+    default: break;
     }
 
   AutoMutex L(m_lock);
@@ -294,6 +295,8 @@ Kumu::LogEntry::TestFilter(i32_t filter) const
        return false;
       break;
 
+    default:
+      break;
     }
 
  return true;
index abfe32b1b77225280e30eb9a484fd82fb3121c40..eee4635afa020b8ee72ef7184eaa2965b7399d9e 100755 (executable)
@@ -94,6 +94,8 @@ class h__ParserState
        case ST_EXT:
          m_State = ST_SEQ;
          return RESULT_OK;
+       default:
+         break;
        }
       
       DefaultLogSink().Error("SEQ follows %s\n", StringParserState(m_State));
@@ -110,6 +112,8 @@ class h__ParserState
        case ST_EXT:
          m_State = ST_SLICE;
          return RESULT_OK;
+       default:
+         break;
        }
       
       DefaultLogSink().Error("Slice follows %s\n", StringParserState(m_State));
@@ -128,6 +132,8 @@ class h__ParserState
        case ST_EXT:
          m_State = ST_PIC;
          return RESULT_OK;
+       default:
+         break;
        }
       
       DefaultLogSink().Error("PIC follows %s\n", StringParserState(m_State));
@@ -144,6 +150,8 @@ class h__ParserState
       case ST_SEQ:
        m_State = ST_GOP;
        return RESULT_OK;
+      default:
+       break;
       }
     
     DefaultLogSink().Error("GOP follows %s\n", StringParserState(m_State));
@@ -161,6 +169,8 @@ class h__ParserState
        case ST_GOP:
          m_State = ST_EXT;
          return RESULT_OK;
+        default:
+         break;
       }
 
     DefaultLogSink().Error("EXT follows %s\n", StringParserState(m_State));
index 6f507c8e9b6b715606ac2d8a56bd783131313696..5d326f2d111a5cf0b53aa1d53f9a1383a9966dc6 100755 (executable)
@@ -133,6 +133,9 @@ class WavFileWriter
          file_count = m_ADesc.ChannelCount / 2;
          m_ChannelCount = 2;
          break;
+
+       default:
+         break;
        }
       assert(file_count && m_ChannelCount);
 
index 2ce033251fa84b23086ba788030fe13a642ea857..6e848440c81cb12caa4940d7d6bba12a45334ee9 100755 (executable)
@@ -593,6 +593,9 @@ main(int argc, const char **argv)
     case MMT_VAL_WRITE:
       result = ReadValidateWriteLargeFile(Options);
       break;
+
+    default:
+      break;
     }
 
   if ( result != RESULT_OK )