commit 3438a9383c769baf4e2f7854f948c40ccc02c91b
author Alessio Bazzica <alessiob@webrtc.org> Wed Oct 14 12:47:50 2020 +0200
committer Commit Bot <commit-bot@chromium.org> Wed Oct 14 12:18:57 2020 +0000
tree fabb5c8ecb4c77e0a87b2f12900640462f4f48e9
parent 44f749dac63c925f156a718496e7425ce3557e4f

AGC2 config change detection fixed

AGC2 is correctly (re)initialized when its config changes.
This CL also improves the `AudioProcessingImpl::ApplyConfig`
readability by defining operator!= also for the AGC1 config.

Bug: webrtc:7494
Change-Id: I62068de32c941e6b18d4618c656f569647042345
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/187120
Commit-Queue: Alessio Bazzica <alessiob@webrtc.org>
Reviewed-by: Per Åhgren <peah@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#32402}

modules/audio_processing/include/audio_processing.h

diff --git a/modules/audio_processing/include/audio_processing.h b/modules/audio_processing/include/audio_processing.h
index d09e2ba..e85ac0c 100644
--- a/modules/audio_processing/include/audio_processing.h
+++ b/modules/audio_processing/include/audio_processing.h
@@ -274,6 +274,11 @@
     // HAL.
     // Recommended to be enabled on the client-side.
     struct GainController1 {
+      bool operator==(const GainController1& rhs) const;
+      bool operator!=(const GainController1& rhs) const {
+        return !(*this == rhs);
+      }
+
       bool enabled = false;
       enum Mode {
         // Adaptive mode intended for use if an analog volume control is
@@ -338,6 +343,11 @@
     // first applies a fixed gain. The adaptive digital AGC can be turned off by
     // setting |adaptive_digital_mode=false|.
     struct GainController2 {
+      bool operator==(const GainController2& rhs) const;
+      bool operator!=(const GainController2& rhs) const {
+        return !(*this == rhs);
+      }
+
       enum LevelEstimator { kRms, kPeak };
       bool enabled = false;
       struct {