Chromium Code Reviews| 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 |