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

Commit 18c225d0 authored by Glenn Kasten's avatar Glenn Kasten
Browse files

Revert "soundpool: reuse channel for same sample if available" 

This reverts commit 570576b5.

Bug: 19635137
Change-Id: I8385ac4c151a375c19c3606698081c947b6216e3
parent b801f733

media/jni/soundpool/SoundPool.cpp

+33 −50
Original line number Diff line number Diff line
@@ -256,7 +256,7 @@ int SoundPool::play(int sampleID, float leftVolume, float rightVolume, 
    dump();



    // allocate a channel

    channel = allocateChannel_l(priority, sampleID);

    channel = allocateChannel_l(priority);



    // no channel allocated - return 0

    if (!channel) {
@@ -271,25 +271,13 @@ int SoundPool::play(int sampleID, float leftVolume, float rightVolume, 
    return channelID;

}



SoundChannel* SoundPool::allocateChannel_l(int priority, int sampleID)

SoundChannel* SoundPool::allocateChannel_l(int priority)

{

    List<SoundChannel*>::iterator iter;

    SoundChannel* channel = NULL;



    // check if channel for given sampleID still available

    // allocate a channel

    if (!mChannels.empty()) {

        for (iter = mChannels.begin(); iter != mChannels.end(); ++iter) {

            if (sampleID == (*iter)->getPrevSampleID() && (*iter)->state() == SoundChannel::IDLE) {

                channel = *iter;

                mChannels.erase(iter);

                ALOGV("Allocated recycled channel for same sampleID");

                break;

            }

        }

    }



    // allocate any channel

    if (!channel && !mChannels.empty()) {

        iter = mChannels.begin();

        if (priority >= (*iter)->priority()) {

            channel = *iter;
@@ -660,7 +648,6 @@ error: 
void SoundChannel::init(SoundPool* soundPool)

{

    mSoundPool = soundPool;

    mPrevSampleID = -1;

}



// call with sound pool lock held
@@ -669,7 +656,7 @@ void SoundChannel::play(const sp<Sample>& sample, int nextChannelID, float leftV 
{

    sp<AudioTrack> oldTrack;

    sp<AudioTrack> newTrack;

    status_t status = NO_ERROR;

    status_t status;



    { // scope for the lock

        Mutex::Autolock lock(&mLock);
@@ -714,7 +701,6 @@ void SoundChannel::play(const sp<Sample>& sample, int nextChannelID, float leftV 
        }

#endif



        if (!mAudioTrack.get() || mPrevSampleID != sample->sampleID()) {

        // mToggle toggles each time a track is started on a given channel.

        // The toggle is concatenated with the SoundChannel address and passed to AudioTrack

        // as callback user data. This enables the detection of callbacks received from the old
@@ -740,15 +726,13 @@ void SoundChannel::play(const sp<Sample>& sample, int nextChannelID, float leftV 
            ALOGE("Error creating AudioTrack");

            goto exit;

        }

        ALOGV("setVolume %p", newTrack.get());

        newTrack->setVolume(leftVolume, rightVolume);

        newTrack->setLoop(0, frameCount, loop);



        // From now on, AudioTrack callbacks received with previous toggle value will be ignored.

        mToggle = toggle;

        mAudioTrack = newTrack;

        } else {

            newTrack = mAudioTrack;

            ALOGV("reusing track %p for sample %d", mAudioTrack.get(), sample->sampleID());

        }

        newTrack->setVolume(leftVolume, rightVolume);

        newTrack->setLoop(0, frameCount, loop);

        mPos = 0;

        mSample = sample;

        mChannelID = nextChannelID;
@@ -891,7 +875,6 @@ bool SoundChannel::doStop_l() 
        setVolume_l(0, 0);

        ALOGV("stop");

        mAudioTrack->stop();

        mPrevSampleID = mSample->sampleID();

        mSample.clear();

        mState = IDLE;

        mPriority = IDLE_PRIORITY;

media/jni/soundpool/SoundPool.h

+1 −3
Original line number Diff line number Diff line
@@ -136,7 +136,6 @@ public: 
    void nextEvent();

    int nextChannelID() { return mNextEvent.channelID(); }

    void dump();

    int getPrevSampleID(void) { return mPrevSampleID; }



private:

    static void callback(int event, void* user, void *info);
@@ -153,7 +152,6 @@ private: 
    int                 mAudioBufferSize;

    unsigned long       mToggle;

    bool                mAutoPaused;

    int                 mPrevSampleID;

};



// application object for managing a pool of sounds
@@ -195,7 +193,7 @@ private: 
    sp<Sample> findSample(int sampleID) { return mSamples.valueFor(sampleID); }

    SoundChannel* findChannel (int channelID);

    SoundChannel* findNextChannel (int channelID);

    SoundChannel* allocateChannel_l(int priority, int sampleID);

    SoundChannel* allocateChannel_l(int priority);

    void moveToFront_l(SoundChannel* channel);

    void notify(SoundPoolEvent event);

    void dump();