Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 355a423d authored by Andy Hung's avatar Andy Hung Committed by Automerger Merge Worker
Browse files

Merge "EffectChain: fix removeEffect_l for global aux effects" am: d38bd2ac... 

Merge "EffectChain: fix removeEffect_l for global aux effects" am: d38bd2ac am: ba9f1d4b am: 09e5ee9b

Original change: https://android-review.googlesource.com/c/platform/frameworks/av/+/2607729



Change-Id: Idd75f2718ae44dc2103f947e498b4ef8936331e3
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 5444bb4b 09e5ee9b

services/audioflinger/Effects.cpp

+2 −1
Original line number Diff line number Diff line
@@ -2456,7 +2456,8 @@ size_t AudioFlinger::EffectChain::removeEffect_l(const sp<EffectModule>& effect, 


            // make sure the input buffer configuration for the new first effect in the chain

            // is updated if needed (can switch from HAL channel mask to mixer channel mask)

            if (i == 0 && size > 1) {

            if (type != EFFECT_FLAG_TYPE_AUXILIARY // TODO(b/284522658) breaks for aux FX, why?

                    && i == 0 && size > 1) {

                mEffects[0]->configure();

                mEffects[0]->setInBuffer(mInBuffer);

                mEffects[0]->updateAccessMode();      // reconfig if neeeded.