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>
Thu, 14 Jan 2016 23:01:38 +0000 (23:01 +0000)
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 922288873f113045e9e2b41c2c204ae2f5326de4..55a32c66fc3461a3a98196ff6ae981ef718f6850 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 73ef4efd5bfffc5e1264f12cd81a6fdf45c08790..02b82f3776f4626084cd03b64f0ed6a36d5e65cd 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 0513d2f6a6e2d060ab1e805a6ad48def2bfb022c..2f2d319ffef0b8c3765e4e392262e5793ecac8f8 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 dc7bf3f2d6733e30b367eea516e3b05efa8b9bbf..af468f278882263cce1101636cdef7d6fccc7217 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 5d3ae0eefeeb568436693f2b2b9711187d35ccf6..ba9a4925f9527a1a41ff05812490ce7587e0b41b 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 763930bb38eac851cad9169885b8159f8474d63b..9c1e528a59287222713102d77793d794fb39f653 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 989030d04f99d66d5cd4c0fa92a384d3df466ec3..80da39be7d7a9acaaa26661f7d80f4f0f778a794 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 )