[FFmpeg-devel] [PATCH] specify a name/description per stream

[Michael Niedermayer]

On Wed, Aug 27, 2008 at 08:04:13PM +0200, Aurelien Jacobs wrote:
[...]
> > > [...]
> > >  
> > > Index: libavformat/avformat.h
> > > ===================================================================
> > > --- libavformat/avformat.h	(revision 14980)
> > > +++ libavformat/avformat.h	(working copy)
> > > @@ -409,6 +409,8 @@
> > >       * - decoding: Set by libavformat.
> > >       */
> > >      AVRational sample_aspect_ratio;
> > > +
> > > +    char *name;  /**< name/description of the track */
> > >  } AVStream;
> > 
> > Maybe char name[1024] is preferrable, like all other metadata in
> > AVFormatContext ?
> 
> I tried to be consistent with other fields in AVStream, such as
> filename. If an array is really preferred I can change it.
> Any other opinion about this.

a more generic metadata API may be interresting
Especially with user specified keys instead of "name" "author", ...
the current system would break down.


[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I have never wished to cater to the crowd; for what I know they do not
approve, and what they approve I do not know. -- Epicurus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080827/cc45729d/attachment.pgp>