[FFmpeg-devel] [PATCH 2/2] avfilter/avfilter: Make ff_tlog_ref() static

Andreas Rheinhardt andreas.rheinhardt at outlook.com
Sat Oct 2 09:25:16 EEST 2021


It allows compilers to inline the one and only call to this function
in its callers or even to optimize it away completely (this function
is empty in case TRACE is not defined).

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at outlook.com>
---
 libavfilter/avfilter.c | 4 ++--
 libavfilter/internal.h | 2 --
 2 files changed, 2 insertions(+), 4 deletions(-)

diff --git a/libavfilter/avfilter.c b/libavfilter/avfilter.c
index d70db1b283..7362bcdab5 100644
--- a/libavfilter/avfilter.c
+++ b/libavfilter/avfilter.c
@@ -47,7 +47,7 @@
 #include "libavutil/ffversion.h"
 const char av_filter_ffversion[] = "FFmpeg version " FFMPEG_VERSION;
 
-void ff_tlog_ref(void *ctx, AVFrame *ref, int end)
+static void tlog_ref(void *ctx, AVFrame *ref, int end)
 {
     ff_tlog(ctx,
             "ref[%p buf:%p data:%p linesize[%d, %d, %d, %d] pts:%"PRId64" pos:%"PRId64,
@@ -1018,7 +1018,7 @@ fail:
 int ff_filter_frame(AVFilterLink *link, AVFrame *frame)
 {
     int ret;
-    FF_TPRINTF_START(NULL, filter_frame); ff_tlog_link(NULL, link, 1); ff_tlog(NULL, " "); ff_tlog_ref(NULL, frame, 1);
+    FF_TPRINTF_START(NULL, filter_frame); ff_tlog_link(NULL, link, 1); ff_tlog(NULL, " "); tlog_ref(NULL, frame, 1);
 
     /* Consistency checks */
     if (link->type == AVMEDIA_TYPE_VIDEO) {
diff --git a/libavfilter/internal.h b/libavfilter/internal.h
index e7c154aff0..116d5fcc96 100644
--- a/libavfilter/internal.h
+++ b/libavfilter/internal.h
@@ -230,8 +230,6 @@ void ff_command_queue_pop(AVFilterContext *filter);
 
 char *ff_get_ref_perms_string(char *buf, size_t buf_size, int perms);
 
-void ff_tlog_ref(void *ctx, AVFrame *ref, int end);
-
 void ff_tlog_link(void *ctx, AVFilterLink *link, int end);
 
 /**
-- 
2.30.2



More information about the ffmpeg-devel mailing list