Add SafeClamp(), which accepts args of different types

webrtc/modules/audio_processing/aec3/suppression_filter.cc

 /*
  *  Copyright (c) 2017 The WebRTC project authors. All Rights Reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  */
 
 #include "webrtc/modules/audio_processing/aec3/suppression_filter.h"
 
 #include <math.h>
 #include <algorithm>
 #include <cstring>
 #include <functional>
 #include <numeric>
 
+#include "webrtc/base/safe_minmax.h"
 #include "webrtc/modules/audio_processing/utility/ooura_fft.h"
 
 namespace webrtc {
 namespace {
 
 // Hanning window from Matlab command win = sqrt(hanning(128)).
 const float kSqrtHanning[kFftLength] = {
     0.00000000000000f, 0.02454122852291f, 0.04906767432742f, 0.07356456359967f,
     0.09801714032956f, 0.12241067519922f, 0.14673047445536f, 0.17096188876030f,
     0.19509032201613f, 0.21910124015687f, 0.24298017990326f, 0.26671275747490f,
@@ skipping 86 matching lines @@
   fft_.Ifft(E, &e_extended);
   std::transform(e_output_old_[0].begin(), e_output_old_[0].end(),
                  std::begin(kSqrtHanning) + kFftLengthBy2, (*e)[0].begin(),
                  [&](float a, float b) { return kIfftNormalization * a * b; });
   std::transform(e_extended.begin(), e_extended.begin() + kFftLengthBy2,
                  std::begin(kSqrtHanning), e_extended.begin(),
                  [&](float a, float b) { return kIfftNormalization * a * b; });
   std::transform((*e)[0].begin(), (*e)[0].end(), e_extended.begin(),
                  (*e)[0].begin(), std::plus<float>());
   std::for_each((*e)[0].begin(), (*e)[0].end(), [](float& x_k) {
-    x_k = std::max(std::min(x_k, 32767.0f), -32768.0f);
+    x_k = rtc::SafeClamp(x_k, -32768.f, 32767.f);
   });
   std::copy(e_extended.begin() + kFftLengthBy2, e_extended.begin() + kFftLength,
             std::begin(e_output_old_[0]));
 
   if (e->size() > 1) {
     // Form time-domain high-band noise.
     std::array<float, kFftLength> time_domain_high_band_noise;
     std::transform(comfort_noise_high_band.re.begin(),
                    comfort_noise_high_band.re.end(), E.re.begin(),
                    [&](float a) { return kIfftNormalization * a; });
@@ skipping 11 matching lines @@
         (*e)[1].begin(), [&](float a, float b) {
           return std::max(
               std::min(b * high_bands_noise_scaling + high_bands_gain * a,
                        32767.0f),
               -32768.0f);
         });
 
     if (e->size() > 2) {
       RTC_DCHECK_EQ(3, e->size());
       std::for_each((*e)[2].begin(), (*e)[2].end(), [&](float& a) {
-        a = std::max(std::min(a * high_bands_gain, 32767.0f), -32768.0f);
+        a = rtc::SafeClamp(a * high_bands_gain, -32768.f, 32767.f);
       });
     }
 
     std::array<float, kFftLengthBy2> tmp;
     for (size_t k = 1; k < e->size(); ++k) {
       std::copy((*e)[k].begin(), (*e)[k].end(), tmp.begin());
       std::copy(e_output_old_[k].begin(), e_output_old_[k].end(),
                 (*e)[k].begin());
       std::copy(tmp.begin(), tmp.end(), e_output_old_[k].begin());
     }
   }
 }
 
 }  // namespace webrtc