[FFmpeg-devel] [PATCH] Clip-Wrapped MXF support (attempt #4)
Maksym Veremeyenko
verem
Thu Jan 27 09:29:00 CET 2011
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.
a already updated patch #0004:
/.../
+ { {
0x06,0x0E,0x2B,0x34,0x04,0x01,0x01,0x01,0x0D,0x01,0x03,0x01,0x02,0x02,0x7F,0x01
}, 16, CODEC_ID_DVVIDEO, Frame }, /* MXF-GC Frame-wrapped DV-based
UndefinedSource */
+ { {
0x06,0x0E,0x2B,0x34,0x04,0x01,0x01,0x01,0x0D,0x01,0x03,0x01,0x02,0x02,0x7F,0x02
}, 16, CODEC_ID_DVVIDEO, Clip }, /* MXF-GC Clip-wrapped DV-based
UndefinedSource */
/.../
anyway, if some UID been lost its easy to add code line...
>
>> 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?
will fix
--
________________________________________
Maksym Veremeyenko
More information about the ffmpeg-devel
mailing list