From 691b447c53e312e0785ff6fd6a858dec309dc952 Mon Sep 17 00:00:00 2001 From: Sergey Silkin Date: Fri, 21 Apr 2023 13:00:14 +0200 Subject: [PATCH] Fix returns from IsSameSettings and IsSameRate in codec tests MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Swap true/false. Bug: webrtc:14852 Change-Id: Id82c0180d33bfc4e5237f4800c3e89fe8d17693f Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/302381 Commit-Queue: Sergey Silkin Reviewed-by: Åsa Persson Cr-Commit-Position: refs/heads/main@{#39917} --- modules/video_coding/codecs/test/video_codec_test.cc | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/modules/video_coding/codecs/test/video_codec_test.cc b/modules/video_coding/codecs/test/video_codec_test.cc index bc4efa0a6f..af52f45ff4 100644 --- a/modules/video_coding/codecs/test/video_codec_test.cc +++ b/modules/video_coding/codecs/test/video_codec_test.cc @@ -84,17 +84,17 @@ struct EncodingSettings { bool IsSameSettings(const EncodingSettings& other) const { if (scalability_mode != other.scalability_mode) { - return true; + return false; } for (auto [layer_id, layer] : layer_settings) { const auto& other_layer = other.layer_settings.at(layer_id); if (layer.resolution != other_layer.resolution) { - return true; + return false; } } - return false; + return true; } bool IsSameRate(const EncodingSettings& other) const { @@ -102,11 +102,11 @@ struct EncodingSettings { const auto& other_layer = other.layer_settings.at(layer_id); if (layer.bitrate != other_layer.bitrate || layer.framerate != other_layer.framerate) { - return true; + return false; } } - return false; + return true; } };