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

James Almer jamrial at gmail.com
Mon Mar 10 17:19:29 EET 2025


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 position of the entry of a given
>> type.
>>
>> Signed-off-by: Leandro Santiago <leandrosansilva at gmail.com>
>> ---
>>   libavutil/frame.c     | 10 ++++++++++
>>   libavutil/frame.h     | 43 ++++++++++++++++++++++++++++++++++++++++---
>>   libavutil/side_data.c | 14 +++++++++++---
>>   3 files changed, 61 insertions(+), 6 deletions(-)
>>
> Is this really such an important usecase that we need to add new public
> functions for this?

IMO, no. There's only one side data type with the multi prop, SEI 
Unregistered, so this function is somewhat overkill.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: OpenPGP_signature.asc
Type: application/pgp-signature
Size: 495 bytes
Desc: OpenPGP digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20250310/99734ea3/attachment.sig>


More information about the ffmpeg-devel mailing list