[FFmpeg-devel] [PATCHv2 1/5] avcodec: add side_data type for updated metadata

Clément Bœsch u at pkh.me
Tue Oct 29 10:17:16 CET 2013


On Fri, Oct 25, 2013 at 10:18:04PM -0400, Ben Boeckel wrote:
> This type is intended to be used to allow codecs to pass updated
> metadata to applications.
> 
> Signed-off-by: Ben Boeckel <mathstuf at gmail.com>
> ---
>  libavcodec/avcodec.h | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h
> index b64331d..f78475e 100644
> --- a/libavcodec/avcodec.h
> +++ b/libavcodec/avcodec.h
> @@ -1012,6 +1012,12 @@ enum AVPacketSideDataType {
>       * follow the timestamp specifier of a WebVTT cue.
>       */
>      AV_PKT_DATA_WEBVTT_SETTINGS,
> +
> +    /**
> +     * A list of zero terminated key/value strings. There is no end marker for
> +     * the list, so it is required to rely on the side data size to stop.
> +     */
> +    AV_PKT_DATA_METADATA_UPDATE,
>  };
>  

This can probably be squashed with the first commit making use of it.
Appart from this, a small entry in doc/APIChanges is required, as well as
a minor (micro?) bump in libavcodec/version.h.

BTW, about this patchset, it would be nice to have an internal test case
about that API; could you make use of it in ffplay (for example) to
display when the metadata change?

-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 490 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20131029/ca78e32b/attachment.asc>


More information about the ffmpeg-devel mailing list