[MPlayer-dev-eng] [PATCH] fix reordering

[Alexander Strange]

On Jul 18, 2010, at 2:30 AM, Dan Oscarsson wrote:

> sön 2010-07-11 klockan 12:02 +0200 skrev Reimar Döffinger:
>> On Sun, Jul 11, 2010 at 11:06:39AM +0200, Dan Oscarsson wrote:
>>>> Also, I really can't see what it should change, correct_pts is not
>>>> set for demux_ts and AFAIK it does not work with it.
>>> 
>>> It is necessary to set -correct-pts for all LATM DVB-T recordings I have
>>> and work fine without my patch - if you do not seek in the file. If you
>>> seek, the reorder code fails - normally missing one out of four frames.
>>> With my patch it work all the time.
>>> 
>>> If you do not like to fix it the way my patch does - what shall we do to
>>> fix it? I do not want to try to understand the seek code in demux_ts.c
>>> to get the seek to work, when it can be fixed so easily in dec_video.c.
>> 
>> First someone could test if it actually does break someting or not (PAFF
>> with -demuxer lavf).
> 
> Do you have a PAFF sample that I could try it with?

http://samples.mplayerhq.hu/V-codecs/h264/PAFF/Grey.ts

If you're touching timestamp code, try this too:
http://samples.mplayerhq.hu/archive/all/mpeg%2bmpeg2video%2bac3%2b%2bnon_monotone_timestamps.mpg

though I don't know if mplayer traditionally had problems with that one.