[FFmpeg-devel] [PATCH] avutil/frame: Add av_frame_transfer_side_data() function

Anton Khirnov anton at khirnov.net
Tue Dec 7 12:28:50 EET 2021


Quoting Hendrik Leppkes (2021-12-07 10:04:47)
> On Tue, Dec 7, 2021 at 9:00 AM Anton Khirnov <anton at khirnov.net> wrote:
> >
> > Quoting Lynne (2021-12-06 20:12:31)
> > > 6 Dec 2021, 19:37 by anton at khirnov.net:
> > >
> > >
> > > I think av_frame_copy_side_data() makes more sense. Transfer
> > > implies that it strips the source frame of side data.
> >
> > The whole point is that it makes refs, not copies (except when it
> > doesn't). I am open to alternative names, but copy is a bad one IMO.
> 
> How about staying close to avframe, and using av_frame_side_data_ref
> or such (analogous to av_frame_ref)?

Possible I guess, but it still has the same problem that it doesn't
always create refs, only if you don't pass the relevant flag.

-- 
Anton Khirnov


More information about the ffmpeg-devel mailing list