[FFmpeg-devel] [PATCH] Clip-Wrapped MXF support (attempt #4)

[Tomas Härdin]

Maksym Veremeyenko skrev 2011-01-25 12:38:
> Maksym Veremeyenko ???????(??):
>> Hi,
>>
>> This is another attempt to submit patches for clip-wrapped MXF files
>> support (generated by Panasonic's P2 camera).
>
> updated patches attached.
>
> most arguable patch is #0007, please review and comment.
>
> PS
> may be its possible to commit patches 0001-0006 that do not change
> functionality and just prepare for Clip-Wrapped support?

That's up to Baptiste. I'd just like to re-state that (re?)introducing 
MXFContainerUL (0001, 0002, 0004) makes mxf_essence_container_uls 
needlessly huge. It might also make samples that worked previously stop 
working - for instance the DV case I mentioned earlier.

> From c121f906a36d580c22b8ba98ae0aa14d9dfd635b Mon Sep 17 00:00:00 2001
> From: Maksym Veremeyenko <verem at m1stereo.tv>
> Date: Tue, 25 Jan 2011 13:28:51 +0200
> Subject: [PATCH 7/7] Clip wrapped code
>
> ---
>  libavformat/mxfdec.c |   57 +++++++++++++++++++++++++++++++++++++++++++++++++-
>  1 files changed, 56 insertions(+), 1 deletions(-)
>
> diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c
> +
> +            /* check for clip wrapped and single track */

Reword the comment since it's no longer checking for single tracks?

> +            if (((MXFTrack*)s->streams[index]->priv_data)->wrapping == Clip) {

/Tomas