[MPlayer-dev-eng] [PATCH] make AVI -forceidx search inside chunks > 1 MB

[Reimar Döffinger]

Hello,
On Sat, Oct 27, 2007 at 01:06:42PM +0200, Alex Beregszaszi wrote:
> On Sat, 2007-10-27 at 09:40 +0200, Reimar Döffinger wrote:
> > attached patch would change avi demuxer behaviour to not skip any AVI
> > chunks > 1 MB but instead search inside those for stream data as well.
> > This admittedly is somewhat hackish (esp. due to the fixed 1 MB limit),
> > but it helps _a lot_ with playing partial or broken files, before
> > -forceidx would usually just stop at the first damage, now it usually
> > recognizes the whole file.
> > Ok to apply? Should the 1 MB be made an option or changed to some other
> > value?
> 
> What about a raw rgb stream?

Well, it would not necessarily break just because the frames are bigger
than 1 MB, if that's your question.
Either way, raw rgb streams seem rare to me even compared to the number
of broken files (and -forceidx probably is very rarely used then because
it will take ages with such huge files), so I doubt they are worth any
extra effort (esp. a extra option), do you think they are?

Greetings,
Reimar Döffinger