From e44f24e1992330ffbdf71950a1a31f61183a408b Mon Sep 17 00:00:00 2001 From: Evan Shrubsole Date: Tue, 9 Feb 2021 17:47:52 +0100 Subject: [PATCH] Handle longer AudioSendStream::Config strings Switch to using StringBuilder which suports a variable sized buffer. Bug: webrtc:12455 Change-Id: I956d2385e6a26ce6fbb73869506d9d79de786a2e Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/206473 Commit-Queue: Evan Shrubsole Reviewed-by: Henrik Andreassson Cr-Commit-Position: refs/heads/master@{#33215} --- call/audio_send_stream.cc | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/call/audio_send_stream.cc b/call/audio_send_stream.cc index 76480f2362..9d25b77ba6 100644 --- a/call/audio_send_stream.cc +++ b/call/audio_send_stream.cc @@ -27,8 +27,7 @@ AudioSendStream::Config::Config(Transport* send_transport) AudioSendStream::Config::~Config() = default; std::string AudioSendStream::Config::ToString() const { - char buf[1024]; - rtc::SimpleStringBuilder ss(buf); + rtc::StringBuilder ss; ss << "{rtp: " << rtp.ToString(); ss << ", rtcp_report_interval_ms: " << rtcp_report_interval_ms; ss << ", send_transport: " << (send_transport ? "(Transport)" : "null"); @@ -39,8 +38,8 @@ std::string AudioSendStream::Config::ToString() const { ss << ", has_dscp: " << (has_dscp ? "true" : "false"); ss << ", send_codec_spec: " << (send_codec_spec ? send_codec_spec->ToString() : ""); - ss << '}'; - return ss.str(); + ss << "}"; + return ss.Release(); } AudioSendStream::Config::Rtp::Rtp() = default;