[FFmpeg-devel] [PATCH v8 09/14] avcodec: add frame side data array to AVCodecContext
James Almer
jamrial at gmail.com
Tue Mar 12 23:26:50 EET 2024
On 3/11/2024 5:58 PM, Jan Ekström wrote:
> This allows configuring an encoder by using AVFrameSideData.
> ---
> libavcodec/avcodec.h | 8 ++++++++
> libavcodec/options.c | 2 ++
> 2 files changed, 10 insertions(+)
>
> diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h
> index 21fc74707f..432a3fd153 100644
> --- a/libavcodec/avcodec.h
> +++ b/libavcodec/avcodec.h
> @@ -2062,6 +2062,14 @@ typedef struct AVCodecContext {
> * Number of entries in side_data_prefer_packet.
> */
> unsigned nb_side_data_prefer_packet;
> +
> + /**
> + * Set containing static side data, such as HDR10 CLL / MDCV structures.
> + * - encoding: set by user
> + * - decoding: unused
This is not taking into account the requests from the last review round.
> + */
> + AVFrameSideData **frame_side_data;
> + int nb_frame_side_data;
> } AVCodecContext;
>
> /**
> diff --git a/libavcodec/options.c b/libavcodec/options.c
> index dcc67e497a..29b961411e 100644
> --- a/libavcodec/options.c
> +++ b/libavcodec/options.c
> @@ -176,6 +176,8 @@ void avcodec_free_context(AVCodecContext **pavctx)
> av_freep(&avctx->inter_matrix);
> av_freep(&avctx->rc_override);
> av_channel_layout_uninit(&avctx->ch_layout);
> + av_frame_side_data_free(
> + &avctx->frame_side_data, &avctx->nb_frame_side_data);
>
> av_freep(pavctx);
> }
More information about the ffmpeg-devel
mailing list