[FFmpeg-cvslog] fftools/ffmpeg_filter: use av_buffer_replace() to improve code

Anton Khirnov git at videolan.org
Mon May 8 11:49:36 EEST 2023


ffmpeg | branch: master | Anton Khirnov <anton at khirnov.net> | Sun Apr 30 12:32:32 2023 +0200| [c26a6c5a523032873dc8bc7fe467b15fdcaa93ea] | committer: Anton Khirnov

fftools/ffmpeg_filter: use av_buffer_replace() to improve code

It is shorter and more efficient.

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=c26a6c5a523032873dc8bc7fe467b15fdcaa93ea
---

 fftools/ffmpeg_filter.c | 10 +++-------
 1 file changed, 3 insertions(+), 7 deletions(-)

diff --git a/fftools/ffmpeg_filter.c b/fftools/ffmpeg_filter.c
index 34c51c23d9..0165be8f77 100644
--- a/fftools/ffmpeg_filter.c
+++ b/fftools/ffmpeg_filter.c
@@ -1415,7 +1415,9 @@ static int ifilter_parameters_from_frame(InputFilter *ifilter, const AVFrame *fr
     AVFrameSideData *sd;
     int ret;
 
-    av_buffer_unref(&ifp->hw_frames_ctx);
+    ret = av_buffer_replace(&ifp->hw_frames_ctx, frame->hw_frames_ctx);
+    if (ret < 0)
+        return ret;
 
     ifilter->format = frame->format;
 
@@ -1433,12 +1435,6 @@ static int ifilter_parameters_from_frame(InputFilter *ifilter, const AVFrame *fr
     if (sd)
         ifp->displaymatrix = av_memdup(sd->data, sizeof(int32_t) * 9);
 
-    if (frame->hw_frames_ctx) {
-        ifp->hw_frames_ctx = av_buffer_ref(frame->hw_frames_ctx);
-        if (!ifp->hw_frames_ctx)
-            return AVERROR(ENOMEM);
-    }
-
     return 0;
 }
 



More information about the ffmpeg-cvslog mailing list