[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