From 5c5b7b38bacc165d6dcf56f337774c625f7453ab Mon Sep 17 00:00:00 2001 From: Tommi Date: Tue, 18 Apr 2023 12:04:58 +0200 Subject: [PATCH] Reduce log spam This removes frequent output (typically 3 times in a row): "RED codec red is missing an associated payload type." Bug: none Change-Id: Ie7e0f344209cb01f9730960a6fec9d5987eaadfd Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/301720 Commit-Queue: Tomas Gunnarsson Reviewed-by: Danil Chapovalov Cr-Commit-Position: refs/heads/main@{#39885} --- pc/media_session.cc | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/pc/media_session.cc b/pc/media_session.cc index 23a2ad11cb..274d7ce8b2 100644 --- a/pc/media_session.cc +++ b/pc/media_session.cc @@ -948,8 +948,10 @@ static const C* GetAssociatedCodecForRed(const std::vector& codec_list, std::string fmtp; if (!red_codec.GetParam(kCodecParamNotInNameValueFormat, &fmtp)) { // Normal for video/RED. - RTC_LOG(LS_WARNING) << "RED codec " << red_codec.name - << " is missing an associated payload type."; + if constexpr (std::is_same_v) { + RTC_LOG(LS_WARNING) << "RED codec " << red_codec.name + << " is missing an associated payload type."; + } return nullptr; }