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

[Dan Oscarsson]

On 2010-07-01 at 18:43 +0200 Reimar Döffinger wrote:
> On Sat, Jun 26, 2010 at 08:44:02AM +0200, Dan Oscarsson wrote:
> > When seeking some demuxers (for example demux_ts.c) returns
> > MP_NOPTS_VALUE for first video frame after the seek. For the mpeg-ts
> > files I have -correct-pts hate to be set for them to play correctly.
> > Unfortunately the reordering code in libmpcodecs/dec_video.c ignores a
> > frame, if the pts have value MP_NOPTS_VALUE, resulting in one frame
> > being out of order in each set of frames after a seek. This gives bad
> > video playback.
> > 
> > As I have written in the patch, I cannot see that including
> > MP_NOPTS_VALUE in the reordering should be a problem, as it is a large
> > negative value (at least on my computers), and sorts before the
> > following frames.
> 
> I suspect that it will break PAFF samples with -demuxer lavf again.
> 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.

Most mpeg-ts files I and a friend of mine have, requires -correct-pts
option to work. My friend records from DVB-T broadcasts and have a lot
of those files. demux_ts works fine with -correct-pts, the mpeg-ts files
we have do not work without it. Without the patch, mplayer cannot play
those mpeg-ts files correctly.

We do not use -demuxer lavf for mpeg-ts as it do not support LATM and
also produce worse audio pts values for many files.

   Dan