Re: [FFmpeg-devel] [PATCH 1/3] avutil/frame: add av_frame_replace

2021-08-13 Thread Andreas Rheinhardt
James Almer: > On 8/13/2021 12:27 PM, Andreas Rheinhardt wrote: >> James Almer: >>> Signed-off-by: James Almer >>> --- >>> Not going to bother with implementing replace for side data, since >>> it's IMO not >>> worth it, but patches are welcome of course. >>> >>> Missing version bump and APIchange

Re: [FFmpeg-devel] [PATCH 1/3] avutil/frame: add av_frame_replace

2021-08-13 Thread James Almer
On 8/13/2021 12:27 PM, Andreas Rheinhardt wrote: James Almer: Signed-off-by: James Almer --- Not going to bother with implementing replace for side data, since it's IMO not worth it, but patches are welcome of course. Missing version bump and APIchanges entry. libavutil/frame.c | 98 +++

Re: [FFmpeg-devel] [PATCH 1/3] avutil/frame: add av_frame_replace

2021-08-13 Thread Andreas Rheinhardt
James Almer: > Signed-off-by: James Almer > --- > Not going to bother with implementing replace for side data, since it's IMO > not > worth it, but patches are welcome of course. > > Missing version bump and APIchanges entry. > > libavutil/frame.c | 98 +

Re: [FFmpeg-devel] [PATCH 1/3] avutil/frame: add av_frame_replace

2021-08-13 Thread James Almer
On 8/13/2021 12:22 PM, Andreas Rheinhardt wrote: James Almer: On 8/10/2021 6:59 PM, James Almer wrote: Signed-off-by: James Almer --- Not going to bother with implementing replace for side data, since it's IMO not worth it, but patches are welcome of course. Missing version bump and APIchange

Re: [FFmpeg-devel] [PATCH 1/3] avutil/frame: add av_frame_replace

2021-08-13 Thread Andreas Rheinhardt
James Almer: > On 8/10/2021 6:59 PM, James Almer wrote: >> Signed-off-by: James Almer >> --- >> Not going to bother with implementing replace for side data, since >> it's IMO not >> worth it, but patches are welcome of course. >> >> Missing version bump and APIchanges entry. >> >>   libavutil/fram

Re: [FFmpeg-devel] [PATCH 1/3] avutil/frame: add av_frame_replace

2021-08-13 Thread James Almer
On 8/10/2021 6:59 PM, James Almer wrote: Signed-off-by: James Almer --- Not going to bother with implementing replace for side data, since it's IMO not worth it, but patches are welcome of course. Missing version bump and APIchanges entry. libavutil/frame.c | 98 +

[FFmpeg-devel] [PATCH 1/3] avutil/frame: add av_frame_replace

2021-08-10 Thread James Almer
Signed-off-by: James Almer --- Not going to bother with implementing replace for side data, since it's IMO not worth it, but patches are welcome of course. Missing version bump and APIchanges entry. libavutil/frame.c | 98 +++ libavutil/frame.h | 18 +