commit 40e36fac615e39dfedb628d0afed853d1eea91a4
author Mikhail Naganov <mnaganov@google.com> Wed Oct 10 14:40:54 2018 -0700
committer Mikhail Naganov <mnaganov@google.com> Wed Oct 10 14:40:54 2018 -0700
tree b67505048c489b00afdc7166c14a73d4b297672a
parent e33b74a91a99d5049291c5e1c37ef2c60cde2a07

audio: Add a missing break to EffectsFactory::getAllDescriptors

Would the legacy HAL return -ENOENT while retrieving an effect,
the enumeration has to be aborted, and the effects queried so far
need to be returned. A missing 'break' was causing returning
no effects at all.

Bug: 117567478
Test: make
Change-Id: I7cc05004f88b39272d06d2ed33a0693f8e1e3d04

audio/effect/all-versions/default/include/effect/all-versions/default/EffectsFactory.impl.h

diff --git a/audio/effect/all-versions/default/include/effect/all-versions/default/EffectsFactory.impl.h b/audio/effect/all-versions/default/include/effect/all-versions/default/EffectsFactory.impl.h
index f27c739..f3256f1 100644
--- a/audio/effect/all-versions/default/include/effect/all-versions/default/EffectsFactory.impl.h
+++ b/audio/effect/all-versions/default/include/effect/all-versions/default/EffectsFactory.impl.h
@@ -101,6 +101,7 @@
                 case -ENOENT: {
                     // No more effects available.
                     result.resize(i);
+                    break;
                 }
                 default: {
                     result.resize(0);