From 06901cfb045dca2d1d2accc51cb133b59247531c Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Oskar=20Segersv=C3=A4rd?= Date: Mon, 17 Feb 2020 17:49:04 +0100 Subject: [PATCH] Use absl::c_any_of instead of a manual for-loop to finding an active encoding MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Bug: webrtc:11319 Change-Id: I00eff8dd1d595570b9b2798a27514ec16fde4bf0 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/168646 Reviewed-by: Henrik Boström Reviewed-by: Ilya Nikolaevskiy Commit-Queue: Oskar Segersvärd Cr-Commit-Position: refs/heads/master@{#30538} --- media/engine/webrtc_video_engine.cc | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/media/engine/webrtc_video_engine.cc b/media/engine/webrtc_video_engine.cc index b1c2828de8..f642c4cc6c 100644 --- a/media/engine/webrtc_video_engine.cc +++ b/media/engine/webrtc_video_engine.cc @@ -2148,11 +2148,9 @@ void WebRtcVideoChannel::WebRtcVideoSendStream::UpdateSendState() { // SVC is used. // The only present simulcast layer should be active if any of the // configured SVC layers is active. - bool is_active = false; - for (size_t i = 0; i < rtp_parameters_.encodings.size(); ++i) { - is_active |= rtp_parameters_.encodings[i].active; - } - active_layers[0] = is_active; + active_layers[0] = + absl::c_any_of(rtp_parameters_.encodings, + [](const auto& encoding) { return encoding.active; }); } // This updates what simulcast layers are sending, and possibly starts // or stops the VideoSendStream.