From fc50110df6091f68ffda621a8427610131392bd8 Mon Sep 17 00:00:00 2001 From: Jonas Olsson Date: Fri, 15 Jun 2018 14:24:10 +0200 Subject: [PATCH] Remove stringstreams from modules/video_coding/ Bug: webrtc:8982 Change-Id: I89dc5c0ccc2a7b69596a1d040f488f47751b20a9 Reviewed-on: https://webrtc-review.googlesource.com/82860 Commit-Queue: Jonas Olsson Reviewed-by: Stefan Holmer Cr-Commit-Position: refs/heads/master@{#23631} --- modules/video_coding/packet_buffer.cc | 19 +++++++------------ 1 file changed, 7 insertions(+), 12 deletions(-) diff --git a/modules/video_coding/packet_buffer.cc b/modules/video_coding/packet_buffer.cc index dc96dcfd2f..ef48ef0e8f 100644 --- a/modules/video_coding/packet_buffer.cc +++ b/modules/video_coding/packet_buffer.cc @@ -12,7 +12,6 @@ #include #include -#include #include #include "common_video/h264/h264_common.h" @@ -349,17 +348,13 @@ std::vector> PacketBuffer::FindFrames( if (is_h264) { // Warn if this is an unsafe frame. if (has_h264_idr && (!has_h264_sps || !has_h264_pps)) { - std::stringstream ss; - ss << "Received H.264-IDR frame " - << "(SPS: " << has_h264_sps << ", PPS: " << has_h264_pps << "). "; - if (sps_pps_idr_is_h264_keyframe_) { - ss << "Treating as delta frame since " - "WebRTC-SpsPpsIdrIsH264Keyframe is enabled."; - } else { - ss << "Treating as key frame since " - "WebRTC-SpsPpsIdrIsH264Keyframe is disabled."; - } - RTC_LOG(LS_WARNING) << ss.str(); + RTC_LOG(LS_WARNING) + << "Received H.264-IDR frame " + << "(SPS: " << has_h264_sps << ", PPS: " << has_h264_pps + << "). Treating as " + << (sps_pps_idr_is_h264_keyframe_ ? "delta" : "key") + << " frame since WebRTC-SpsPpsIdrIsH264Keyframe is " + << (sps_pps_idr_is_h264_keyframe_ ? "enabled." : "disabled"); } // Now that we have decided whether to treat this frame as a key frame