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

[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).
Then the need to use -correct-pts should be fixed.
Then of course the missing timestamps from the ts demuxer should be
corrected, since that is going to cause issues either way.
Then the -correct-pts code should be investigated why it fails to recover
even after a longer time and if that can be fixed/autodetected.
Lastly, the lavf demuxer should finally be changed to export LATM streams,
that ffmpeg can't handle them is just a ridiculous excuse to stop everyone
else from even seeing them.