[FFmpeg-soc] [Patch]Mxf muxer 0.0.1

[Baptiste Coudurier]

Baptiste Coudurier wrote:
> Michael Niedermayer wrote:
>> On Thu, Aug 14, 2008 at 11:41:01AM +0800, zhentan feng wrote:
>>> Hi
>>>
>>> 2008/8/14 Michael Niedermayer <michaelni at gmx.at>
>>>
>>>> On Wed, Aug 13, 2008 at 03:47:15PM +0800, zhentan feng wrote:
>>>>> Hi,
>>>>>
>>>>> after changing the code in svn soc according to former reviews,
>>>>> here is the diff between svn soc and ffmpeg trunk.
>>>>>
>>>>> mainly do this works:
>>>>> 1) add mxfenc.c to impliment MXF simple muxer
>>>>> 2) extract common code from mxfenc.c and mxfdec.c into mxf.h
>>>>>
>>>>> --
>>>>> Best wishes~
>>>>> Index: libavformat/mxfenc.c
>>>>> ===================================================================
>>>>> --- libavformat/mxfenc.c      (revision 0)
>>>>> +++ libavformat/mxfenc.c      (revision 0)
>>>>> @@ -0,0 +1,1058 @@
>>>>> +/*
>>>>> + * MXF muxer
>>>>> + * Copyright (c) 2008 GUCAS, Zhentan Feng<spyfeng at gmail dot com>
>>>>> + *
>>>>> + * This file is part of FFmpeg.
>>>>> + *
>>>>> + * FFmpeg is free software; you can redistribute it and/or
>>>>> + * modify it under the terms of the GNU Lesser General Public
>>>>> + * License as published by the Free Software Foundation; either
>>>>> + * version 2.1 of the License, or (at your option) any later version.
>>>>> + *
>>>>> + * FFmpeg is distributed in the hope that it will be useful,
>>>>> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
>>>>> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
>>>>> + * Lesser General Public License for more details.
>>>>> + *
>>>>> + * You should have received a copy of the GNU Lesser General Public
>>>>> + * License along with FFmpeg; if not, write to the Free Software
>>>>> + * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA
>>>> 02110-1301 USA
>>>>> + */
>>>>> +
>>>>> +/*
>>>>> + * References
>>>>> + * SMPTE 336M KLV Data Encoding Protocol Using Key-Length-Value
>>>>> + * SMPTE 377M MXF File Format Specifications
>>>>> + * SMPTE 379M MXF Generic Container
>>>>> + * SMPTE 381M Mapping MPEG Streams into the MXF Generic Container
>>>>> + * SMPTE RP210: SMPTE Metadata Dictionary
>>>>> + * SMPTE RP224: Registry of SMPTE Universal Labels
>>>>> + */
>>>>> +
>>>> ok
>>>> (this part and all other ok-ed parts can be commited
>>>>  that way the patch will become smaller and easier to review)
>>>>
>>> I diff the files with FFmpeg trunk.
>>> It seems that I can not commit to FFmpeg repo, but I'll improved it in soc
>>> svn.
>> baptiste (or any othetr volunteer), can you commit the ok-ed parts?
>>
> 
> I will asap, when I figure out how many hunks can be commited, except a
> few structs definitions and license header ;)
> 
> Stopping the kidding, yes I will apply them asap.
> 
> [...]
> 

Applied ok'd hunks in mxfenc.c

-- 
Baptiste COUDURIER                              GnuPG Key Id: 0x5C1ABAAA
Smartjog USA Inc.                                http://www.smartjog.com
Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA