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

Commit a8356f66 authored by Glenn Kasten's avatar Glenn Kasten
Browse files

Simplify RecordTrack::stop() 

Change-Id: Ib959c1e9dc9544d12277ce11bea445118b2e0521
parent fc38a2e0

services/audioflinger/Threads.cpp

+2 −1
Original line number Diff line number Diff line
@@ -4031,8 +4031,9 @@ void AudioFlinger::RecordThread::handleSyncStartEvent(const sp<SyncEvent>& event 
    }

}



bool AudioFlinger::RecordThread::stop_l(RecordThread::RecordTrack* recordTrack) {

bool AudioFlinger::RecordThread::stop(RecordThread::RecordTrack* recordTrack) {

    ALOGV("RecordThread::stop");

    AutoMutex _l(mLock);

    if (recordTrack != mActiveTrack.get() || recordTrack->mState == TrackBase::PAUSING) {

        return false;

    }

services/audioflinger/Threads.h

+1 −1
Original line number Diff line number Diff line
@@ -747,7 +747,7 @@ public: 


            // ask the thread to stop the specified track, and

            // return true if the caller should then do it's part of the stopping process

            bool        stop_l(RecordTrack* recordTrack);

            bool        stop(RecordTrack* recordTrack);



            void        dump(int fd, const Vector<String16>& args);

            AudioStreamIn* clearInput();

services/audioflinger/Tracks.cpp

+1 −6
Original line number Diff line number Diff line
@@ -1656,12 +1656,7 @@ void AudioFlinger::RecordThread::RecordTrack::stop() 
    sp<ThreadBase> thread = mThread.promote();

    if (thread != 0) {

        RecordThread *recordThread = (RecordThread *)thread.get();

        recordThread->mLock.lock();

        bool doStop = recordThread->stop_l(this);

        if (doStop) {

        }

        recordThread->mLock.unlock();

        if (doStop) {

        if (recordThread->stop(this)) {

            AudioSystem::stopInput(recordThread->id());

        }

    }