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

[Dan Oscarsson]

sön 2010-07-18 klockan 14:17 +0200 skrev Reimar Döffinger:
> On Sun, Jul 18, 2010 at 01:23:44PM +0200, Dan Oscarsson wrote:
> > sön 2010-07-18 klockan 12:55 +0200 skrev Reimar Döffinger:
> > > On Sun, Jul 18, 2010 at 11:30:05AM +0200, Dan Oscarsson wrote:
> > > > > Then the need to use -correct-pts should be fixed.
> > > > 
> > > > You mean that demuxer/decoder should do the reordering instead?
> > > 
> > > There is no timestamp reordering without -correct-pts since the
> > > timestamps aren't used. Only frame rate is used.
> > 
> > The A-V sync correction code in mplayer.c are using timstamps to
> > calculate how to correct a-v sync. So timestamp reordering in
> > decode_video would reduce corrections even when -correct-pts is not
> > used.
> 
> But it is supposed to work even when they aren't right.
> If it doesn't work right there's probably some other bug, and
> just hiding it is not a good solution.

It does work, but works better if timestamps are in correct order as a-v
correction gets smaller. So why not always reorder?

   Dan