commit 0a981d36cf96307d837a2a8167dfbdfb91b55d4e
author Shraddha Basantwani <shraddha.basantwani@ittiam.com> Wed Jul 24 08:42:50 2024 +0000
committer Shraddha Basantwani <shraddha.basantwani@ittiam.com> Tue Jul 30 05:23:02 2024 +0000
tree 99eb93142ed75add42a5e5831b8ebba8a0b0ee35
parent 3935a3fa5de8e00a7f5ca3360c2bf09d07126d52

Audio Effect : Add base implementation of commandImpl method

Removing code repeatation and moving this method to base class.
Effect implementations can override enable, disable and reset methods.

Bug: 305866207
Test: atest --test-mapping hardware/interfaces/audio/aidl/vts:presubmit
Change-Id: Iceb7ad51185d6f6bd2b6d70a5a1dd51e7bd37276

audio/aidl/vts/EffectHelper.h

diff --git a/audio/aidl/vts/EffectHelper.h b/audio/aidl/vts/EffectHelper.h
index cad1195..0fa170f 100644
--- a/audio/aidl/vts/EffectHelper.h
+++ b/audio/aidl/vts/EffectHelper.h
@@ -397,10 +397,10 @@
                                                               outputBuffer.size(), outputBuffer));
         }
 
+        // Disable the process
         ASSERT_NO_FATAL_FAILURE(command(effect, CommandId::STOP));
         EXPECT_NO_FATAL_FAILURE(EffectHelper::readFromFmq(statusMQ, 0, outputMQ, 0, outputBuffer));
 
-        // Disable the process
         ASSERT_NO_FATAL_FAILURE(command(effect, CommandId::RESET));
     }