Fix missing-break-fallthrough warning.
Adds a break; after RTC_NOTREACHED(). Also removes default case to catch
if any other codec type is added.
BUG=
R=stefan@webrtc.org
Review URL: https://codereview.webrtc.org/1905573002 .
Cr-Commit-Position: refs/heads/master@{#12443}
diff --git a/webrtc/modules/video_coding/packet_buffer.cc b/webrtc/modules/video_coding/packet_buffer.cc
index 8cbfc0c..d7b4f44 100644
--- a/webrtc/modules/video_coding/packet_buffer.cc
+++ b/webrtc/modules/video_coding/packet_buffer.cc
@@ -207,25 +207,22 @@
VideoCodecType codec_type = data_buffer_[start_index].codec;
switch (codec_type) {
- case kVideoCodecULPFEC :
- case kVideoCodecRED :
- case kVideoCodecUnknown : {
+ case kVideoCodecULPFEC:
+ case kVideoCodecRED:
+ case kVideoCodecUnknown:
RTC_NOTREACHED();
- }
- case kVideoCodecVP8 : {
+ break;
+ case kVideoCodecVP8:
ManageFrameVp8(std::move(frame));
break;
- }
- case kVideoCodecVP9 : {
+ case kVideoCodecVP9:
// TODO(philipel): ManageFrameVp9(std::move(frame));
break;
- }
- case kVideoCodecH264 :
- case kVideoCodecI420 :
- case kVideoCodecGeneric :
- default : {
+ case kVideoCodecH264:
+ case kVideoCodecI420:
+ case kVideoCodecGeneric:
ManageFrameGeneric(std::move(frame));
- }
+ break;
}
}