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

Commit fb246971 authored by Vishal Bhoj's avatar Vishal Bhoj Committed by codeworkx
Browse files

Revert "Don't believe the PV decoder's lies..." 

We see corruption in timestamp and hence framedrop
while playing an MPEG4 file. Reverting this commit
fixes the issue.

This reverts commit 97770292.

Change-Id: Ie7feda68ac203c82f66d7edcd23af14cc9df02e9
parent 12be7037

media/libstagefright/codecs/m4v_h263/dec/SoftMPEG4.cpp

+1 −6
Original line number Diff line number Diff line
@@ -421,13 +421,8 @@ void SoftMPEG4::onQueueFilled(OMX_U32 portIndex) { 


        int32_t bufferSize = inHeader->nFilledLen;



        // The PV decoder is lying to us, sometimes it'll claim to only have

        // consumed a subset of the buffer when it clearly consumed all of it.

        // ignore whatever it says...

        int32_t tmp = bufferSize;



        if (PVDecodeVideoFrame(

                    mHandle, &bitstream, &timestamp, &tmp,

                    mHandle, &bitstream, &timestamp, &bufferSize,

                    &useExtTimestamp,

                    outHeader->pBuffer) != PV_TRUE) {

            LOGE("failed to decode video frame.");