[FFmpeg-soc] MXF muxer version 0.0.4

[zhentan feng]

Hi
2008/8/24 Michael Niedermayer <michaelni at gmx.at>

> On Sat, Aug 23, 2008 at 06:23:16PM +0200, Baptiste Coudurier wrote:
> > 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 ?
>
> TypeBottom? Iam not happy about it either, i just gave up after
> "Sequence" and 0xf0 being used in its place. It didnt appear important
> enough to reject the code and i had some communication problems with
> zhentan
>
>
sorry for my poor english :(
I mean, as Michael said writing uid   by mxf_write_uuid(uuid_base, type,
value);
we use enum value as the type naturally,
but Mataerial package and sourcepackage both have Trak references, then it
wil confilict with mxf_write_uuid(uuid_base, Track, value).

therefore,  we need unique type for different Tracks, in order to avoid add
Track1, Track2, TrackN in MXFMetadataType, I tried to 2 ways to solve this:

1, use a big number as offset.
write(uuid_base, Track + 0xf0, value)

2, use a flag type called TypeBottom which always in the bottom of
MXFMetadataType.
write(uuid_base, Track + TypeBottom, value)

either of them is not a good solution, but Michael seems accept the later.

>
> [...]
>
>


-- 
Best wishes~