[MPlayer-dev-eng] [PATCH] reorder pts when needed

Dan Oscarsson Dan.Oscarsson at tieto.com
Sat Nov 26 17:00:29 CET 2011


fre 2011-11-25 klockan 08:07 +0100 skrev Dan Oscarsson:
> tor 2011-11-24 klockan 19:21 +0100 skrev Reimar Döffinger:
> > Just fixing the timestamps for one specific problematic case is more
> > hiding the issue than fixing it, even if it makes sense as an additional
> > improvement to a real fix.

Looking at the code I see that MP_IMGTYPE_INCOMPLETE is only used for
h264 decoding by ffmpeg, so my code might just fix the h264 case. Though
I am not sure exactly what cases different demuxer/decoders return value
for the reorder routine to use.

Mostly what I have done is to enable reorder for more cases then just
correct_pts. Attached is an alternativ way to do the patch that may look
a bit cleaner.

I think it could be even simpler by allowing MP_NOPTS_VALUE to be
reordered and I suspect the limiting to delay values in the end could be
replaced by simpler code in the beginning.

   Dan
-------------- next part --------------
A non-text attachment was scrubbed...
Name: reorder-alt.patch
Type: text/x-patch
Size: 1605 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20111126/c37943e8/attachment.bin>


More information about the MPlayer-dev-eng mailing list