From d7c6f32ddff4eef7fc1cf933edae35582dbc3edf Mon Sep 17 00:00:00 2001 From: Sham Rathod Date: Mon, 26 Dec 2022 11:17:10 +0530 Subject: [PATCH] Audio AIDL : Add default value to params Bug: 258124419 Test: atest VtsHalBassBoostTargetTest Test: atest VtsHalVirtualizerTargetTest Test: atest VtsHalVisualizerTargetTest Test: atest VtsHalAECTargetTest Test: atest VtsHalAGCTargetTest Test: atest VtsHalNSTargetTest Change-Id: Iad68cf01f8a27dfe9ba17584f2c3b43f0d586d86 --- .../default/acousticEchoCanceler/AcousticEchoCancelerSw.h | 2 +- .../default/automaticGainControl/AutomaticGainControlSw.h | 7 ++++--- audio/aidl/default/bassboost/BassBoostSw.h | 2 +- audio/aidl/default/noiseSuppression/NoiseSuppressionSw.h | 2 +- audio/aidl/default/virtualizer/VirtualizerSw.h | 2 +- audio/aidl/default/visualizer/VisualizerSw.h | 2 +- 6 files changed, 9 insertions(+), 8 deletions(-) diff --git a/audio/aidl/default/acousticEchoCanceler/AcousticEchoCancelerSw.h b/audio/aidl/default/acousticEchoCanceler/AcousticEchoCancelerSw.h index 809812a9cc..5f1bc46fa4 100644 --- a/audio/aidl/default/acousticEchoCanceler/AcousticEchoCancelerSw.h +++ b/audio/aidl/default/acousticEchoCanceler/AcousticEchoCancelerSw.h @@ -37,7 +37,7 @@ class AcousticEchoCancelerSwContext final : public EffectContext { int getEchoDelay() const { return mEchoDelayUs; } private: - int mEchoDelayUs; + int mEchoDelayUs = 0; }; class AcousticEchoCancelerSw final : public EffectImpl { diff --git a/audio/aidl/default/automaticGainControl/AutomaticGainControlSw.h b/audio/aidl/default/automaticGainControl/AutomaticGainControlSw.h index e23fec0cc7..2724835e91 100644 --- a/audio/aidl/default/automaticGainControl/AutomaticGainControlSw.h +++ b/audio/aidl/default/automaticGainControl/AutomaticGainControlSw.h @@ -41,9 +41,10 @@ class AutomaticGainControlSwContext final : public EffectContext { int getSaturationMargin(); private: - int mDigitalGain; - AutomaticGainControl::LevelEstimator mLevelEstimator; - int mSaturationMargin; + int mDigitalGain = 0; + AutomaticGainControl::LevelEstimator mLevelEstimator = + AutomaticGainControl::LevelEstimator::RMS; + int mSaturationMargin = 0; }; class AutomaticGainControlSw final : public EffectImpl { diff --git a/audio/aidl/default/bassboost/BassBoostSw.h b/audio/aidl/default/bassboost/BassBoostSw.h index 411b9c99ea..91086e9848 100644 --- a/audio/aidl/default/bassboost/BassBoostSw.h +++ b/audio/aidl/default/bassboost/BassBoostSw.h @@ -46,7 +46,7 @@ class BassBoostSwContext final : public EffectContext { int getBbStrengthPm() const { return mStrength; } private: - int mStrength; + int mStrength = 0; }; class BassBoostSw final : public EffectImpl { diff --git a/audio/aidl/default/noiseSuppression/NoiseSuppressionSw.h b/audio/aidl/default/noiseSuppression/NoiseSuppressionSw.h index 0c03038226..f39d8e5768 100644 --- a/audio/aidl/default/noiseSuppression/NoiseSuppressionSw.h +++ b/audio/aidl/default/noiseSuppression/NoiseSuppressionSw.h @@ -37,7 +37,7 @@ class NoiseSuppressionSwContext final : public EffectContext { NoiseSuppression::Level getLevel(); private: - NoiseSuppression::Level mLevel; + NoiseSuppression::Level mLevel = NoiseSuppression::Level::LOW; }; class NoiseSuppressionSw final : public EffectImpl { diff --git a/audio/aidl/default/virtualizer/VirtualizerSw.h b/audio/aidl/default/virtualizer/VirtualizerSw.h index 05974a8fed..b4482da909 100644 --- a/audio/aidl/default/virtualizer/VirtualizerSw.h +++ b/audio/aidl/default/virtualizer/VirtualizerSw.h @@ -45,7 +45,7 @@ class VirtualizerSwContext final : public EffectContext { int getVrStrength() const { return mStrength; } private: - int mStrength; + int mStrength = 0; }; class VirtualizerSw final : public EffectImpl { diff --git a/audio/aidl/default/visualizer/VisualizerSw.h b/audio/aidl/default/visualizer/VisualizerSw.h index edaf1232fe..3c9e6d283c 100644 --- a/audio/aidl/default/visualizer/VisualizerSw.h +++ b/audio/aidl/default/visualizer/VisualizerSw.h @@ -80,7 +80,7 @@ class VisualizerSwContext final : public EffectContext { int mCaptureSize = kMaxCaptureSize; Visualizer::ScalingMode mScalingMode = Visualizer::ScalingMode::NORMALIZED; Visualizer::MeasurementMode mMeasurementMode = Visualizer::MeasurementMode::NONE; - int mLatency; + int mLatency = 0; const Visualizer::GetOnlyParameters::Measurement mMeasurement = {0, 0}; std::vector mCaptureSampleBuffer; };