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

Tomas Härdin tomas.hardin at codemill.se
Sun Dec 18 16:47:54 CET 2011


On Sun, 2011-12-18 at 14:12 +0100, Reimar Döffinger wrote:
> 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

Do you have a sample that demonstrates this problem? If so, then what
software wrote it? The first attempt at a solution would be to file a
bug report with the authors, not encourage the use of such shitty
writers.
In case it's from a hardware writer then things of course get more
interesting. In any case, I oppose adding things like this on a
hypothetical whim.

>(and I am not sure that the index is not used for more than only
>seeking even with the new code).

For OP1a files the index is only used for seeking, to the start of the
desired edit unit.
For OPAtom the index also drives the demuxing. A broken index would
cause lots of headache I find unpleasant to think about. A misssing
index is sometimes possible to deal with though, using a parser.

/Tomas



More information about the ffmpeg-devel mailing list