commit c527a7c2b1bfd26e8f3086e1b653d56e521379d9
author Glenn Kasten <gkasten@google.com> Tue Aug 13 15:43:49 2013 -0700
committer Glenn Kasten <gkasten@google.com> Wed Aug 14 17:18:57 2013 -0700
tree 0946aada0e5cefeebc9fc87ec2d0ae843c5b0553
parent 47c2070b2ce8aedb7300c0aad91caccf3c383841

Pull in scope of effectChains

Change-Id: Id5336f8663099088a0958fb8bd59fc5df9f92829

services/audioflinger/Threads.cpp

diff --git a/services/audioflinger/Threads.cpp b/services/audioflinger/Threads.cpp
index 387ed4e..49520a1 100644
--- a/services/audioflinger/Threads.cpp
+++ b/services/audioflinger/Threads.cpp
@@ -4195,7 +4195,6 @@
 {
     AudioBufferProvider::Buffer buffer;
     sp<RecordTrack> activeTrack;
-    Vector< sp<EffectChain> > effectChains;
 
     nsecs_t lastWarning = 0;
 
@@ -4213,6 +4212,7 @@
 
         processConfigEvents();
 
+        Vector< sp<EffectChain> > effectChains;
         { // scope for mLock
             Mutex::Autolock _l(mLock);
             checkForNewParameters_l();
@@ -4420,7 +4420,7 @@
         }
         // enable changes in effect chain
         unlockEffectChains(effectChains);
-        effectChains.clear();
+        // effectChains doesn't need to be cleared, since it is cleared by destructor at scope end
     }
 
     standby();