admin/webrtc_m130
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix returns from IsSameSettings and IsSameRate in codec tests

Browse Source 
Swap true/false.

Bug: webrtc:14852
Change-Id: Id82c0180d33bfc4e5237f4800c3e89fe8d17693f
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/302381
Commit-Queue: Sergey Silkin <ssilkin@webrtc.org>
Reviewed-by: Åsa Persson <asapersson@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#39917}
This commit is contained in:
Sergey Silkin 2023-04-21 13:00:14 +02:00 committed by WebRTC LUCI CQ
parent ff6cd53303
commit 691b447c53
1 changed files with 5 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
modules/video_coding/codecs/test/video_codec_test.cc
View File

@ -84,17 +84,17 @@ struct EncodingSettings {
bool IsSameSettings(const EncodingSettings& other) const { bool IsSameSettings(const EncodingSettings& other) const {
if (scalability_mode != other.scalability_mode) { if (scalability_mode != other.scalability_mode) {
return true; return false;
} }
for (auto [layer_id, layer] : layer_settings) { for (auto [layer_id, layer] : layer_settings) {
const auto& other_layer = other.layer_settings.at(layer_id); const auto& other_layer = other.layer_settings.at(layer_id);
if (layer.resolution != other_layer.resolution) { if (layer.resolution != other_layer.resolution) {
return true; return false;
} }
} }
return false; return true;
} }
bool IsSameRate(const EncodingSettings& other) const { bool IsSameRate(const EncodingSettings& other) const {
@ -102,11 +102,11 @@ struct EncodingSettings {
const auto& other_layer = other.layer_settings.at(layer_id); const auto& other_layer = other.layer_settings.at(layer_id);
if (layer.bitrate != other_layer.bitrate || if (layer.bitrate != other_layer.bitrate ||
layer.framerate != other_layer.framerate) { layer.framerate != other_layer.framerate) {
return true; return false;
} }
} }
return false; return true;
} }
}; };