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

Commit 3da7f05a authored by Andy Hung's avatar Andy Hung
Browse files

Fix timestretch AV sync 

Allow sonic library to do input/output data management otherwise
it appears to accumulate audio in its buffers.

Bug: 28409398
Change-Id: Ic25915bc844d8416ec8125b58795d09e838f901e
parent 3e5605b8

services/audioflinger/BufferProviders.cpp

+10 −10
Original line number Diff line number Diff line
@@ -474,18 +474,18 @@ void TimestretchBufferProvider::processFrames(void *dstBuffer, size_t *dstFrames 
    ALOGV("processFrames(%zu %zu)  remaining(%zu)", *dstFrames, *srcFrames, mRemaining);

    // Note dstFrames is the required number of frames.



    if (!mAudioPlaybackRateValid) {

        //fallback mode

        // Ensure consumption from src is as expected.

        // TODO: add logic to track "very accurate" consumption related to speed, original sampling

        // rate, actual frames processed.



        const size_t targetSrc = *dstFrames * mPlaybackRate.mSpeed;

        if (*srcFrames < targetSrc) { // limit dst frames to that possible

            *dstFrames = *srcFrames / mPlaybackRate.mSpeed;

        } else if (*srcFrames > targetSrc + 1) {

            *srcFrames = targetSrc + 1;

        }



    if (!mAudioPlaybackRateValid) {

        //fallback mode

        if (*dstFrames > 0) {

            switch(mPlaybackRate.mFallbackMode) {

            case AUDIO_TIMESTRETCH_FALLBACK_CUT_REPEAT: