OSDN Git Service

Remove dead 'if'
authorGlenn Kasten <gkasten@google.com>
Wed, 14 Aug 2013 20:55:45 +0000 (13:55 -0700)
committerGlenn Kasten <gkasten@google.com>
Thu, 15 Aug 2013 17:18:27 +0000 (10:18 -0700)
Change-Id: I23c082f0115ae598cad293be0ac66a130afb692a

services/audioflinger/Threads.cpp

index a8854ca..4471d82 100644 (file)
@@ -4233,46 +4233,44 @@ bool AudioFlinger::RecordThread::threadLoop()
                 acquireWakeLock_l();
                 continue;
             }
-            if (mActiveTrack != 0) {
-                if (mActiveTrack->isTerminated()) {
-                    removeTrack_l(mActiveTrack);
+            if (mActiveTrack->isTerminated()) {
+                removeTrack_l(mActiveTrack);
+                mActiveTrack.clear();
+            } else {
+                switch (mActiveTrack->mState) {
+                case TrackBase::PAUSING:
+                    standby();
                     mActiveTrack.clear();
-                } else {
-                    switch (mActiveTrack->mState) {
-                    case TrackBase::PAUSING:
-                        standby();
+                    mStartStopCond.broadcast();
+                    break;
+
+                case TrackBase::RESUMING:
+                    if (mReqChannelCount != mActiveTrack->channelCount()) {
                         mActiveTrack.clear();
                         mStartStopCond.broadcast();
-                        break;
-
-                    case TrackBase::RESUMING:
-                        if (mReqChannelCount != mActiveTrack->channelCount()) {
+                    } else if (readOnce) {
+                        // record start succeeds only if first read from audio input
+                        // succeeds
+                        if (mBytesRead >= 0) {
+                            mActiveTrack->mState = TrackBase::ACTIVE;
+                        } else {
                             mActiveTrack.clear();
-                            mStartStopCond.broadcast();
-                        } else if (readOnce) {
-                            // record start succeeds only if first read from audio input
-                            // succeeds
-                            if (mBytesRead >= 0) {
-                                mActiveTrack->mState = TrackBase::ACTIVE;
-                            } else {
-                                mActiveTrack.clear();
-                            }
-                            mStartStopCond.broadcast();
                         }
-                        mStandby = false;
-                        break;
-
-                    case TrackBase::ACTIVE:
-                        break;
+                        mStartStopCond.broadcast();
+                    }
+                    mStandby = false;
+                    break;
 
-                    case TrackBase::IDLE:
-                        break;
+                case TrackBase::ACTIVE:
+                    break;
 
-                    default:
-                        LOG_FATAL("Unexpected mActiveTrack->mState %d", mActiveTrack->mState);
-                    }
+                case TrackBase::IDLE:
+                    break;
 
+                default:
+                    LOG_FATAL("Unexpected mActiveTrack->mState %d", mActiveTrack->mState);
                 }
+
             }
             lockEffectChains_l(effectChains);
         }