Re: [FFmpeg-devel] [PATCH] avutil/frame: add new functions to get frame side data

2025-03-11 Thread Andreas Rheinhardt
Leandro Santiago: > The functions are: > > av_frame_side_data_get_nth_c() > av_frame_side_data_get_nth() > av_frame_get_nth_side_data() > > They mimic the counterparts without the `_nth` suffix/infix, with an extra > argument, which specifies the relative position of the entry of a given > type.

Re: [FFmpeg-devel] [PATCH] avutil/frame: add new functions to get frame side data

2025-03-11 Thread James Almer
On 3/10/2025 11:57 AM, Andreas Rheinhardt wrote: Leandro Santiago: The functions are: av_frame_side_data_get_nth_c() av_frame_side_data_get_nth() av_frame_get_nth_side_data() They mimic the counterparts without the `_nth` suffix/infix, with an extra argument, which specifies the relative posit

Re: [FFmpeg-devel] [PATCH] avutil/frame: add new functions to get frame side data

2025-03-10 Thread Leandro Santiago
Just pinging to see if anyone has had time to look at this patch :-) I also noticed that this patch did not trigger an entry on patchwork. Is it normal or should the patches go to patchwork automatically? Also, what's the correct approach within ffmpeg codebase to implement unit tests? Or would

[FFmpeg-devel] [PATCH] avutil/frame: add new functions to get frame side data

2025-03-04 Thread Leandro Santiago
The functions are: av_frame_side_data_get_nth_c() av_frame_side_data_get_nth() av_frame_get_nth_side_data() They mimic the counterparts without the `_nth` suffix/infix, with an extra argument, which specifies the relative position of the entry of a given type. Signed-off-by: Leandro Santiago --