[FFmpeg-soc] [Patch]Mxf muxer 0.0.1

Baptiste Coudurier baptiste.coudurier at smartjog.com
Thu Aug 14 23:42:22 CEST 2008


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.

[...]

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



More information about the FFmpeg-soc mailing list