[FFmpeg-devel] [PATCH] metadata API is now ready to be part of public API

Justin Ruggles justin.ruggles
Sun Mar 1 00:56:21 CET 2009


Aurelien Jacobs wrote:
> Hi,
> 
> I think the new metadata API is now ready to be used in the wild.
> So attached patch makes it officially part of public API.
> Note that all (de)muxers are now using this new API.
> 
> The only step left for this transition is to disable old API for
> next major version, and to document this deprecation.

What do you think about adding a generic api function to append to an
existing entry instead of replacing it?  I'm thinking in terms of
formats like vorbiscomment which explicitly allow duplicate entries for
the same key.  Something like:

int av_metadata_append(AVMetadata **pm, const char *key, const char
*value, const char *separator);

-Justin




More information about the ffmpeg-devel mailing list