zqz/platform-external-webrtc
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix missing-break-fallthrough warning.

Browse Source 
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}
This commit is contained in:
Peter Boström
2016-04-20 14:06:45 +02:00
parent 02b3d275a0
commit 2ddf09397f
1 changed files with 10 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files

23
webrtc/modules/video_coding/packet_buffer.cc
View File

@ -207,25 +207,22 @@ void PacketBuffer::ManageFrame(std::unique_ptr<RtpFrameObject> frame) {
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;
}
}