Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(245)

Unified Diff: webrtc/modules/video_coding/packet_buffer.cc

Issue 1905573002: Fix missing-break-fallthrough warning. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Created 4 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/modules/video_coding/packet_buffer.cc
diff --git a/webrtc/modules/video_coding/packet_buffer.cc b/webrtc/modules/video_coding/packet_buffer.cc
index 8cbfc0c66f417c2cd6063f12ef41cd5d22a909fd..d7b4f449e7be065f1bb2872678f0863f9fe0de56 100644
--- a/webrtc/modules/video_coding/packet_buffer.cc
+++ b/webrtc/modules/video_coding/packet_buffer.cc
@@ -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;
}
}
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698