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

2022-05-23 Thread Soft Works
> -Original Message- > From: ffmpeg-devel On Behalf Of Anton > Khirnov > Sent: Monday, December 6, 2021 7:38 PM > To: ffmpeg-devel@ffmpeg.org > Subject: [FFmpeg-devel] [PATCH] avutil/frame: Add > av_frame_transfer_side_data() function > > From: Sof

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

2021-12-07 Thread Andreas Rheinhardt
Anton Khirnov: > From: Soft Works > > Signed-off-by: softworkz > Signed-off-by: Anton Khirnov > --- > doc/APIchanges | 4 +++ > libavutil/frame.c | 63 ++--- > libavutil/frame.h | 20 ++ > libavutil/version.h | 4 +-- > 4 files cha

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

2021-12-07 Thread Anton Khirnov
Quoting Hendrik Leppkes (2021-12-07 10:04:47) > On Tue, Dec 7, 2021 at 9:00 AM Anton Khirnov wrote: > > > > Quoting Lynne (2021-12-06 20:12:31) > > > 6 Dec 2021, 19:37 by an...@khirnov.net: > > > > > > > > > I think av_frame_copy_side_data() makes more sense. Transfer > > > implies that it strips

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

2021-12-07 Thread Hendrik Leppkes
On Tue, Dec 7, 2021 at 9:00 AM Anton Khirnov wrote: > > Quoting Lynne (2021-12-06 20:12:31) > > 6 Dec 2021, 19:37 by an...@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

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

2021-12-07 Thread Anton Khirnov
Quoting Lynne (2021-12-06 20:12:31) > 6 Dec 2021, 19:37 by an...@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 altern

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

2021-12-06 Thread Lynne
6 Dec 2021, 19:37 by an...@khirnov.net: > From: Soft Works > > Signed-off-by: softworkz > Signed-off-by: Anton Khirnov > --- > doc/APIchanges | 4 +++ > libavutil/frame.c | 63 ++--- > libavutil/frame.h | 20 ++ > libavutil/version.

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

2021-12-06 Thread Soft Works
> -Original Message- > From: ffmpeg-devel On Behalf Of Anton > Khirnov > Sent: Monday, December 6, 2021 7:38 PM > To: ffmpeg-devel@ffmpeg.org > Subject: [FFmpeg-devel] [PATCH] avutil/frame: Add > av_frame_transfer_side_data() function > > From: Sof

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

2021-12-06 Thread Anton Khirnov
From: Soft Works Signed-off-by: softworkz Signed-off-by: Anton Khirnov --- doc/APIchanges | 4 +++ libavutil/frame.c | 63 ++--- libavutil/frame.h | 20 ++ libavutil/version.h | 4 +-- 4 files changed, 63 insertions(+), 28 deletion