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

Tomas Härdin tomas.hardin at codemill.se
Tue Jan 10 15:28:43 CET 2012


On Thu, 2012-01-05 at 22:18 +0100, Reimar Döffinger wrote:
> This allow users to work around regressions introduced by index support
> and helps with debugging.
> 
> Signed-off-by: Reimar Döffinger <Reimar.Doeffinger at gmx.de>
> ---
>  libavformat/mxfdec.c |    3 +++
>  1 files changed, 3 insertions(+), 0 deletions(-)
> 
> diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c
> index 66af2e4..1498ebb 100644
> --- a/libavformat/mxfdec.c
> +++ b/libavformat/mxfdec.c
> @@ -1529,6 +1529,9 @@ static int mxf_parse_handle_essence(MXFContext *mxf)
>          /* remember where we were so we don't end up seeking further back than this */
>          mxf->last_forward_tell = avio_tell(pb);
>  
> +        if (mxf->fc->flags & AVFMT_FLAG_IGNIDX)
> +            return 0;
> +

Uhm, this doesn't make sense. Shouldn't this flag be checked in code
related to index parsing or seeking, like mxf_read_index_table_segment()
or somewhere in mxf_parse_structural_metadata()?

/Tomas



More information about the ffmpeg-devel mailing list