commit 15eb7a6fa9828fb7aed5ea58529763806d1a7c86
author Mikhail Naganov <mnaganov@google.com> Wed Oct 10 14:40:54 2018 -0700
committer Mikhail Naganov <mnaganov@google.com> Thu Oct 11 12:21:27 2018 -0700
tree 78a906d863dbe3438dcd5f3e4939e56a843aaa0b
parent 76c45692bf79861eac818c1406d56648b7912f0f

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
Merged-In: 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 1882a2c..b0351c9 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);