Finalization of the first version of EchoCanceller 3

webrtc/modules/audio_processing/aec3/suppression_gain.h

Index: webrtc/modules/audio_processing/aec3/suppression_gain.h
diff --git a/webrtc/modules/audio_processing/aec3/suppression_gain.h b/webrtc/modules/audio_processing/aec3/suppression_gain.h
new file mode 100644
index 0000000000000000000000000000000000000000..6c2d7f7e837f181407b544daa5ef36e74fed7b0b
--- /dev/null
+++ b/webrtc/modules/audio_processing/aec3/suppression_gain.h
@@ -0,0 +1,63 @@
+/*
+ *  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.
+ */
+
+#ifndef WEBRTC_MODULES_AUDIO_PROCESSING_AEC3_SUPPRESSION_GAIN_H_
+#define WEBRTC_MODULES_AUDIO_PROCESSING_AEC3_SUPPRESSION_GAIN_H_
+
+#include <array>
+#include <vector>

[hlundin-webrtc, 2017/02/21 16:34:03]

Not used.

[peah-webrtc, 2017/02/21 23:00:41]

Done.

+
+#include "webrtc/base/constructormagic.h"
+#include "webrtc/modules/audio_processing/aec3/aec3_common.h"
+#include "webrtc/modules/audio_processing/aec3/fft_buffer.h"
+
+namespace webrtc {
+
+#if defined(WEBRTC_ARCH_X86_FAMILY)
+
+void ComputeGains_SSE2(

[hlundin-webrtc, 2017/02/21 16:34:03]

Why are these two non-member functions exposed in the h-file? For testing?
Consider wrapping them in an extra namespace, e.g., namespace
suppression_gain_helpers, or namespace aec3.
https://google.github.io/styleguide/cppguide.html#Nonmember,_Static_Member,_a...

[peah-webrtc, 2017/02/21 23:00:41]

Yes, these are only there for the sake of the methods being possible to test.

I'll put these in the namespace aec3, as this is something that occurs in other
places as well.

PTAL

[hlundin-webrtc, 2017/02/22 15:24:05]

Good.

[peah-webrtc, 2017/02/22 23:51:36]

Acknowledged.

+    const std::array<float, kFftLengthBy2Plus1>& nearend_power,
+    const std::array<float, kFftLengthBy2Plus1>& residual_echo_power,
+    const std::array<float, kFftLengthBy2Plus1>& comfort_noise_power,
+    float strong_nearend_margin,
+    std::array<float, kFftLengthBy2 - 1>* previous_gain_squared,
+    std::array<float, kFftLengthBy2 - 1>* previous_masker,
+    std::array<float, kFftLengthBy2Plus1>* gain);
+
+#endif
+
+void ComputeGains(
+    const std::array<float, kFftLengthBy2Plus1>& nearend_power,
+    const std::array<float, kFftLengthBy2Plus1>& residual_echo_power,
+    const std::array<float, kFftLengthBy2Plus1>& comfort_noise_power,
+    float strong_nearend_margin,
+    std::array<float, kFftLengthBy2 - 1>* previous_gain_squared,
+    std::array<float, kFftLengthBy2 - 1>* previous_masker,
+    std::array<float, kFftLengthBy2Plus1>* gain);
+
+class SuppressionGain {
+ public:
+  explicit SuppressionGain(Aec3Optimization optimization);
+  void GetGain(const std::array<float, kFftLengthBy2Plus1>& nearend_power,
+               const std::array<float, kFftLengthBy2Plus1>& residual_echo_power,
+               const std::array<float, kFftLengthBy2Plus1>& comfort_noise_power,
+               float strong_nearend_margin,
+               std::array<float, kFftLengthBy2Plus1>* gain);
+
+ private:
+  const Aec3Optimization optimization_;
+  std::array<float, kFftLengthBy2 - 1> previous_gain_squared_;
+  std::array<float, kFftLengthBy2 - 1> previous_masker_;
+  RTC_DISALLOW_IMPLICIT_CONSTRUCTORS(SuppressionGain);
+};
+
+}  // namespace webrtc
+
+#endif  // WEBRTC_MODULES_AUDIO_PROCESSING_AEC3_SUPPRESSION_GAIN_H_