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

Unified Diff: webrtc/modules/rtp_rtcp/source/rtp_utility.cc

Issue 1535963002: Wire-up BWE feedback for audio receive streams. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Fix audio receive stream test expectation. Created 5 years 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
Index: webrtc/modules/rtp_rtcp/source/rtp_utility.cc
diff --git a/webrtc/modules/rtp_rtcp/source/rtp_utility.cc b/webrtc/modules/rtp_rtcp/source/rtp_utility.cc
index fcd43ad197fb0604838e1fd175e52e48e72df2e0..9d35cbc1a774b8032521a375a465efc3f78c8abb 100644
--- a/webrtc/modules/rtp_rtcp/source/rtp_utility.cc
+++ b/webrtc/modules/rtp_rtcp/source/rtp_utility.cc
@@ -386,7 +386,6 @@ void RtpHeaderParser::ParseOneByteExtensionHeader(
<< "RTP extension header 15 encountered. Terminate parsing.";
return;
}
-
RTPExtensionType type;
if (ptrExtensionMap->GetType(id, &type) != 0) {
// If we encounter an unknown extension, just skip over it.
@@ -397,7 +396,7 @@ void RtpHeaderParser::ParseOneByteExtensionHeader(
case kRtpExtensionTransmissionTimeOffset: {
if (len != 2) {
LOG(LS_WARNING) << "Incorrect transmission time offset len: "
- << len;
+ << static_cast<int>(len);
the sun 2015/12/20 23:16:15 Would it be easier to store the length in an int?
stefan-webrtc 2015/12/21 08:01:29 Haha, agreed.
return;
}
// 0 1 2 3
@@ -413,7 +412,8 @@ void RtpHeaderParser::ParseOneByteExtensionHeader(
}
case kRtpExtensionAudioLevel: {
if (len != 0) {
- LOG(LS_WARNING) << "Incorrect audio level len: " << len;
+ LOG(LS_WARNING) << "Incorrect audio level len: "
+ << static_cast<int>(len);
return;
}
// 0 1
@@ -429,7 +429,8 @@ void RtpHeaderParser::ParseOneByteExtensionHeader(
}
case kRtpExtensionAbsoluteSendTime: {
if (len != 2) {
- LOG(LS_WARNING) << "Incorrect absolute send time len: " << len;
+ LOG(LS_WARNING) << "Incorrect absolute send time len: "
+ << static_cast<int>(len);
return;
}
// 0 1 2 3
@@ -446,7 +447,8 @@ void RtpHeaderParser::ParseOneByteExtensionHeader(
case kRtpExtensionVideoRotation: {
if (len != 0) {
LOG(LS_WARNING)
- << "Incorrect coordination of video coordination len: " << len;
+ << "Incorrect coordination of video coordination len: "
+ << static_cast<int>(len);
return;
}
// 0 1
@@ -460,8 +462,8 @@ void RtpHeaderParser::ParseOneByteExtensionHeader(
}
case kRtpExtensionTransportSequenceNumber: {
if (len != 1) {
- LOG(LS_WARNING)
- << "Incorrect peer connection sequence number len: " << len;
+ LOG(LS_WARNING) << "Incorrect transport sequence number len: "
+ << static_cast<int>(len);
return;
}
// 0 1 2

Powered by Google App Engine
This is Rietveld 408576698