commit 2839b850e2020a24bca512d5d337d2ed8b15e9e8
author Shunkai Yao <yaoshunkai@google.com> Sat Feb 03 06:48:10 2024 +0000
committer Shunkai Yao <yaoshunkai@google.com> Mon Feb 05 22:13:21 2024 +0000
tree 63f61b57d2dae3f5b8b1a4e47c12ed33f8d8ba00
parent ada1d9843229a54e9adda12840764c21c83c97bc

Add backward compatibility in effect hal default implementation

Also fix a build warnning

Bug: 322780092
Test: latest libAudioHal with V1 effect HAL
Test: atest VtsHalAudioEffectTargetTest

Change-Id: I38e0302ca162bb3bada2bdd21d948026551a4e00

media/libeffects/visualizer/aidl/Visualizer.cpp

diff --git a/media/libeffects/visualizer/aidl/Visualizer.cpp b/media/libeffects/visualizer/aidl/Visualizer.cpp
index 0303842..fa651a6 100644
--- a/media/libeffects/visualizer/aidl/Visualizer.cpp
+++ b/media/libeffects/visualizer/aidl/Visualizer.cpp
@@ -61,8 +61,8 @@
         MAKE_RANGE(Visualizer, latencyMs, 0, VisualizerContext::kMaxLatencyMs),
         MAKE_RANGE(Visualizer, captureSamples, 0, VisualizerContext::kMaxCaptureBufSize),
         /* get only parameters, set invalid range (min > max) to indicate not support set */
-        MAKE_RANGE(Visualizer, measurement, Visualizer::Measurement({.peak = 1, .rms = 1}),
-                   Visualizer::Measurement({.peak = 0, .rms = 0})),
+        MAKE_RANGE(Visualizer, measurement, Visualizer::Measurement({.rms = 1, .peak = 1}),
+                   Visualizer::Measurement({.rms = 0, .peak = 0})),
         MAKE_RANGE(Visualizer, captureSampleBuffer, std::vector<uint8_t>({1}),
                    std::vector<uint8_t>({0}))};
 const Capability VisualizerImpl::kCapability = {