[FFmpeg-devel] [PATCH] mxfdec: respect AVFMT_FLAG_IGNIDX.

Reimar Döffinger Reimar.Doeffinger at gmx.de
Sun Dec 18 14:12:22 CET 2011


On Sun, Dec 18, 2011 at 01:39:25PM +0100, Tomas Härdin wrote:
> On Fri, 2011-12-16 at 21:01 +0100, Reimar Döffinger wrote:
> > This is useful if either the index is just broken or as a
> > temporary workaround if the new index parsing code doesn't
> > handle some case yet.
> 
> Actually, the new code absoletes this hack. The reason is simple: except
> for OPAtom it works mostly like the old code.

It's not a hack, it is supporting a feature.
And I don't think the new code will obsolete it, if the index contains
complete nonsense the new code will still try to use it for seeking
(and I am not sure that the index is not used for more than only
seeking even with the new code).
Or the really short variant: If an index is optional there's no good
reason to not allow completely disabling it, at least when it's
only one line of code.


More information about the ffmpeg-devel mailing list