[FFmpeg-soc] MXF muxer version 0.0.4

[Baptiste Coudurier]

Michael Niedermayer wrote:
> [...]
>
>>  static void mxf_update_header_partition(AVFormatContext *s, int64_t footer_partition_offset)
>>  {
>>      MXFContext *mxf = s->priv_data;
>> Index: mxf.h
>> ===================================================================
>> --- mxf.h	(revision 14903)
>> +++ mxf.h	(working copy)
>> @@ -41,6 +41,7 @@
>>      Identification,
>>      ContentStorage,
>>      SubDescriptor,
>> +    TypeBottom,// add metadata type before this
>>  };
>>  
>>  typedef struct {
> 
> ok
> 
> 

Why do we need this ?

I don't understand why do we need this for uuid genereration ?

My idea is set a MXFContext->cur_uid to a static start value, then
increment this value for each uid needed.

Is there something wrong with this ? I don't see how this mess (if
MaterialPackage etc ...) is needed.

UID is just a way to reference unique metadata in mxf file, enum seems
not needed, streams[0] and streams[1] does not necessearly need to have
number in sequence.

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