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

[Reimar Döffinger]

On Mon, Dec 19, 2011 at 10:38:42AM +0100, Tomas Härdin wrote:
> > > It could just be that its a partial file, with only part of the
> > index
> > > missing.
> > > Or some transmission error damaging mostly the index.
> > I can see that for MXF it is not going to be very convincing case,
> > but for AVI one of the biggest issues was actually with torrents:
> > They can easily end up with an index that looks almost ok, except
> > that somewhere in the middle there is a block of 0s instead of a
> > proper index.
> 
> I can see that being at least a plausible use case, if unlikely (your
> typical MXF user has enough bandwidth to re-download).
> 
> Anyway: The patch no longer applies. A suitable replacement could
> possibly be to check that flag in mxf_read_seek().

Yes, I mistakenly thought your patches had already applied.
I might check it an make a new suggestion if I have a moment to
spend on it.
Though I think it preferable to check where the index is read,
to e.g. also allow to avoid any seeking/processing overhead
this involves (e.g. as a method to speed up playback startup
when playing over http with high latencies).
So my idea would have been to put it in parallel to
if (!pb->seekable) (but without printing the warning), but
I am not completely sure if this might break something more than
just the index.