Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(117)

Unified Diff: webrtc/modules/audio_processing/aec3/erle_estimator.cc

Issue 2808513003: Add SafeClamp(), which accepts args of different types (Closed)
Patch Set: rebase Created 3 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: webrtc/modules/audio_processing/aec3/erle_estimator.cc
diff --git a/webrtc/modules/audio_processing/aec3/erle_estimator.cc b/webrtc/modules/audio_processing/aec3/erle_estimator.cc
index c1394126a5441da8d3ae6e6da06c976cafb98fbd..7faf22cf603d05fad1d7ac21b17899ee864ed186 100644
--- a/webrtc/modules/audio_processing/aec3/erle_estimator.cc
+++ b/webrtc/modules/audio_processing/aec3/erle_estimator.cc
@@ -12,6 +12,8 @@
#include <algorithm>
+#include "webrtc/base/safe_minmax.h"
+
namespace webrtc {
namespace {
@@ -48,7 +50,7 @@ void ErleEstimator::Update(
if (new_erle > erle_[k]) {
hold_counters_[k - 1] = 100;
erle_[k] += 0.1f * (new_erle - erle_[k]);
- erle_[k] = std::max(kMinErle, std::min(erle_[k], max_erle));
+ erle_[k] = rtc::SafeClamp(erle_[k], kMinErle, max_erle);
}
}
}

Powered by Google App Engine
This is Rietveld 408576698