Quoting Jan Ekström (2024-02-29 17:42:56) > 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 43859251cc..411f4caad3 100644 > --- a/libavcodec/avcodec.h > +++ b/libavcodec/avcodec.h > @@ -2120,6 +2120,14 @@ typedef struct AVCodecContext { > * an error. > */ > int64_t frame_num; > + > + /** > + * Set containing static side data, such as HDR10 CLL / MDCV structures. > + * - encoding: set by user
May be set by the caller before avcodec_open2(). Afterwards owned and freed by the encoder. > + * - decoding: unused > + */ > + AVFrameSideData **frame_side_data; > + int nb_frame_side_data; I don't like calling the field 'frame' side data, because it is not associated with any frame. I'd prefer to call it 'decoded_side_data' or 'raw_side_data', to be consistent with existing 'coded_side_data'. -- Anton Khirnov _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".