projects
/
asdcplib.git
/ commitdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
| commitdiff |
tree
raw
|
patch
|
inline
| side by side (parent:
7861300
)
Fix enum switch/case statements with unhandled values.
author
Carl Hetherington
<cth@carlh.net>
Mon, 11 Jan 2016 10:43:43 +0000
(10:43 +0000)
committer
Carl Hetherington
<cth@carlh.net>
Tue, 3 Dec 2019 15:44:08 +0000
(16:44 +0100)
src/AS_DCP_MPEG2.cpp
patch
|
blob
|
history
src/AS_DCP_PCM.cpp
patch
|
blob
|
history
src/JP2K_Codestream_Parser.cpp
patch
|
blob
|
history
src/KM_log.cpp
patch
|
blob
|
history
src/MPEG2_Parser.cpp
patch
|
blob
|
history
src/WavFileWriter.h
patch
|
blob
|
history
src/kmfilegen.cpp
patch
|
blob
|
history
diff --git
a/src/AS_DCP_MPEG2.cpp
b/src/AS_DCP_MPEG2.cpp
index b7877b54549be8a135afcd77432619bdc55ae2f8..598e585d303607f5095671c31c2d790aaa8038a7 100755
(executable)
--- a/
src/AS_DCP_MPEG2.cpp
+++ b/
src/AS_DCP_MPEG2.cpp
@@
-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;
case FRAME_I: Flags = 0x00; break;
case FRAME_P: Flags = 0x22; break;
case FRAME_B: Flags = 0x33; break;
+ default: break;
}
if ( FrameBuf.GOPStart() )
}
if ( FrameBuf.GOPStart() )
diff --git
a/src/AS_DCP_PCM.cpp
b/src/AS_DCP_PCM.cpp
index d90581440474498fe2fdf7ee15f375b573e88f98..1ca197c4c8a21b916c4b210a90ea14261676ffb4 100755
(executable)
--- a/
src/AS_DCP_PCM.cpp
+++ b/
src/AS_DCP_PCM.cpp
@@
-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;
case PCM::CF_CFG_6:
ADescObj->ChannelAssignment = DefaultSMPTEDict().Type(MDD_DCAudioChannelCfg_MCA).ul;
break;
+
+ default:
+ break;
}
return RESULT_OK;
}
return RESULT_OK;
diff --git
a/src/JP2K_Codestream_Parser.cpp
b/src/JP2K_Codestream_Parser.cpp
index 0ed26492e82e4f8ccce9295d92c3247a219108e0..5401db2e2d5f807f1a1f8cf9ecd2ec044fa47429 100755
(executable)
--- a/
src/JP2K_Codestream_Parser.cpp
+++ b/
src/JP2K_Codestream_Parser.cpp
@@
-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;
memcpy(&PDesc.QuantizationDefault, NextMarker.m_Data, NextMarker.m_DataSize);
PDesc.QuantizationDefault.SPqcdLength = NextMarker.m_DataSize - 1;
break;
+
+ default:
+ break;
}
}
}
}
diff --git
a/src/KM_log.cpp
b/src/KM_log.cpp
index 182d91e6ee2cb2730230cc39672e0d5ad8fbd562..885dd1262a9ac6755dbab1b94ae0b303dde790a3 100755
(executable)
--- a/
src/KM_log.cpp
+++ b/
src/KM_log.cpp
@@
-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;
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);
}
AutoMutex L(m_lock);
@@
-294,6
+295,8
@@
Kumu::LogEntry::TestFilter(i32_t filter) const
return false;
break;
return false;
break;
+ default:
+ break;
}
return true;
}
return true;
diff --git
a/src/MPEG2_Parser.cpp
b/src/MPEG2_Parser.cpp
index abfe32b1b77225280e30eb9a484fd82fb3121c40..eee4635afa020b8ee72ef7184eaa2965b7399d9e 100755
(executable)
--- a/
src/MPEG2_Parser.cpp
+++ b/
src/MPEG2_Parser.cpp
@@
-94,6
+94,8
@@
class h__ParserState
case ST_EXT:
m_State = ST_SEQ;
return RESULT_OK;
case ST_EXT:
m_State = ST_SEQ;
return RESULT_OK;
+ default:
+ break;
}
DefaultLogSink().Error("SEQ follows %s\n", StringParserState(m_State));
}
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;
case ST_EXT:
m_State = ST_SLICE;
return RESULT_OK;
+ default:
+ break;
}
DefaultLogSink().Error("Slice follows %s\n", StringParserState(m_State));
}
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;
case ST_EXT:
m_State = ST_PIC;
return RESULT_OK;
+ default:
+ break;
}
DefaultLogSink().Error("PIC follows %s\n", StringParserState(m_State));
}
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;
case ST_SEQ:
m_State = ST_GOP;
return RESULT_OK;
+ default:
+ break;
}
DefaultLogSink().Error("GOP follows %s\n", StringParserState(m_State));
}
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;
case ST_GOP:
m_State = ST_EXT;
return RESULT_OK;
+ default:
+ break;
}
DefaultLogSink().Error("EXT follows %s\n", StringParserState(m_State));
}
DefaultLogSink().Error("EXT follows %s\n", StringParserState(m_State));
diff --git
a/src/WavFileWriter.h
b/src/WavFileWriter.h
index 6f507c8e9b6b715606ac2d8a56bd783131313696..5d326f2d111a5cf0b53aa1d53f9a1383a9966dc6 100755
(executable)
--- a/
src/WavFileWriter.h
+++ b/
src/WavFileWriter.h
@@
-133,6
+133,9
@@
class WavFileWriter
file_count = m_ADesc.ChannelCount / 2;
m_ChannelCount = 2;
break;
file_count = m_ADesc.ChannelCount / 2;
m_ChannelCount = 2;
break;
+
+ default:
+ break;
}
assert(file_count && m_ChannelCount);
}
assert(file_count && m_ChannelCount);
diff --git
a/src/kmfilegen.cpp
b/src/kmfilegen.cpp
index 2ce033251fa84b23086ba788030fe13a642ea857..6e848440c81cb12caa4940d7d6bba12a45334ee9 100755
(executable)
--- a/
src/kmfilegen.cpp
+++ b/
src/kmfilegen.cpp
@@
-593,6
+593,9
@@
main(int argc, const char **argv)
case MMT_VAL_WRITE:
result = ReadValidateWriteLargeFile(Options);
break;
case MMT_VAL_WRITE:
result = ReadValidateWriteLargeFile(Options);
break;
+
+ default:
+ break;
}
if ( result != RESULT_OK )
}
if ( result != RESULT_OK )