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

Nicolas George george at nsup.org
Tue Jun 7 15:18:42 EEST 2022


James Almer (12022-06-07):
> Signed-off-by: James Almer <jamrial at gmail.com>
> ---
>  libavutil/frame.c | 132 ++++++++++++++++++++++++++++++++++++++++++++++
>  libavutil/frame.h |  13 +++++
>  2 files changed, 145 insertions(+)

I do not like having a new function without anything that uses it. It
results in several cases in a function that is not really useful because
of a small design flaw. Can you update other parts of the code to make
use of the new function?

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20220607/1aaf78cf/attachment.sig>


More information about the ffmpeg-devel mailing list