[MPlayer-cvslog] r35486 - in trunk/libmpdemux: demuxer.c demuxer.h

[Reimar Döffinger]

On Wed, Nov 28, 2012 at 08:37:18PM +0100, Ingo Brückl wrote:
> Reimar Döffinger wrote on Wed, 28 Nov 2012 20:17:20 +0100:
> 
> > On Wed, Nov 28, 2012 at 06:22:18PM +0100, Ingo Brückl wrote:
> >> > Author: reimar
> >> > Date: Mon Nov 26 20:30:58 2012
> >> > New Revision: 35486
> >>
> >> > Log:
> >> > Try to improve seeking in files with only few video packets,
> >> > in particular files with cover art.
> >>
> >> > This might cause issues with badly interleaved files, particularly
> >> > together with -audio-delay, even though I did not see issues
> >> > in my very limited testing.
> >>
> >> It completely breaks playback of some avi files. Sample is on its way.
> 
> > Should be fixed,
> 
> It is, thanks a lot.
> 
> > with some positive side effects like switching faster to non-interleaved
> > mode, which should reduce startup time and improve behaviour when seeking
> > right after startup for such files.
> 
> Though I don't know what disadvantages (if any) of ni playback (which isn't
> the default?) may be, there seem to be no playback issues with the sample
> after that change. It seems to behave and play like it did with a pre-r35486
> MPlayer.

I assumed that it had used NI mode before, did it not?
In that case the advantage now is that we recognize earlier that the
file is not properly interleaved, which would give the advantages I
mentioned.
NI mode can perform worse due to lots of seeking back and forth.