Re: [FFmpeg-devel] [PATCH 04/11] avutil/frame: av_frame_side_data_new_struct()

2025-03-11 Thread Andreas Rheinhardt
James Almer: > As well as the AV_SIDE_DATA_PROP_STRUCT prop, to define types that describe > a fixed-size C struct and not a flat byte array. > This excludes types like VIDEO_ENC_PARAMS as the struct it describes may have > extra bytes allocated past the end of the struct. > > Signed-off-by: James

Re: [FFmpeg-devel] [PATCH 04/11] avutil/frame: av_frame_side_data_new_struct()

2025-03-06 Thread James Almer
On 3/6/2025 11:50 AM, Andreas Rheinhardt wrote: Can't you add a new parameter nb_elems to also handle all the side data that only has a single array? This would then also cover AV_FRAME_DATA_REGIONS_OF_INTEREST, AV_FRAME_DATA_VIDEO_ENC_PARAMS, AV_FRAME_DATA_DETECTION_BBOXES, AV_FRAME_DATA_VIDEO_H

[FFmpeg-devel] [PATCH 04/11] avutil/frame: av_frame_side_data_new_struct()

2025-02-20 Thread James Almer
As well as the AV_SIDE_DATA_PROP_STRUCT prop, to define types that describe a fixed-size C struct and not a flat byte array. This excludes types like VIDEO_ENC_PARAMS as the struct it describes may have extra bytes allocated past the end of the struct. Signed-off-by: James Almer --- libavutil/am