[FFmpeg-cvslog] avfilter/internal: Uninline ff_insert_(in|out)pad()

Andreas Rheinhardt git at videolan.org
Sun Aug 22 19:06:11 EEST 2021


ffmpeg | branch: master | Andreas Rheinhardt <andreas.rheinhardt at outlook.com> | Wed Aug 11 21:29:34 2021 +0200| [dff1709eb2dc6f3f8128abb4e2c764cc7c5fc92b] | committer: Andreas Rheinhardt

avfilter/internal: Uninline ff_insert_(in|out)pad()

These functions are not hot at all.

Reviewed-by: Nicolas George <george at nsup.org>
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at outlook.com>

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

 libavfilter/avfilter.c | 24 +++++++++++++++++++++---
 libavfilter/internal.h | 29 +++--------------------------
 2 files changed, 24 insertions(+), 29 deletions(-)

diff --git a/libavfilter/avfilter.c b/libavfilter/avfilter.c
index 28123655d6..2681d04fc0 100644
--- a/libavfilter/avfilter.c
+++ b/libavfilter/avfilter.c
@@ -101,9 +101,17 @@ void ff_command_queue_pop(AVFilterContext *filter)
     av_free(c);
 }
 
-int ff_append_pad(unsigned *count,
-                   AVFilterPad **pads, AVFilterLink ***links,
-                   AVFilterPad *newpad)
+/**
+ * Append a new pad.
+ *
+ * @param count  Pointer to the number of pads in the list
+ * @param pads   Pointer to the pointer to the beginning of the list of pads
+ * @param links  Pointer to the pointer to the beginning of the list of links
+ * @param newpad The new pad to add. A copy is made when adding.
+ * @return >= 0 in case of success, a negative AVERROR code on error
+ */
+static int append_pad(unsigned *count, AVFilterPad **pads,
+                      AVFilterLink ***links, AVFilterPad *newpad)
 {
     AVFilterLink **newlinks;
     AVFilterPad *newpads;
@@ -126,6 +134,16 @@ int ff_append_pad(unsigned *count,
     return 0;
 }
 
+int ff_append_inpad(AVFilterContext *f, AVFilterPad *p)
+{
+    return append_pad(&f->nb_inputs, &f->input_pads, &f->inputs, p);
+}
+
+int ff_append_outpad(AVFilterContext *f, AVFilterPad *p)
+{
+    return append_pad(&f->nb_outputs, &f->output_pads, &f->outputs, p);
+}
+
 int avfilter_link(AVFilterContext *src, unsigned srcpad,
                   AVFilterContext *dst, unsigned dstpad)
 {
diff --git a/libavfilter/internal.h b/libavfilter/internal.h
index 51b0afa24d..cc95f06c4c 100644
--- a/libavfilter/internal.h
+++ b/libavfilter/internal.h
@@ -230,33 +230,10 @@ void ff_tlog_ref(void *ctx, AVFrame *ref, int end);
 void ff_tlog_link(void *ctx, AVFilterLink *link, int end);
 
 /**
- * Insert a new pad.
- *
- * @param count Pointer to the number of pads in the list
- * @param pads Pointer to the pointer to the beginning of the list of pads
- * @param links Pointer to the pointer to the beginning of the list of links
- * @param newpad The new pad to add. A copy is made when adding.
- * @return >= 0 in case of success, a negative AVERROR code on error
+ * Append a new input/output pad to the filter's list of such pads.
  */
-int ff_append_pad(unsigned *count,
-                   AVFilterPad **pads, AVFilterLink ***links,
-                   AVFilterPad *newpad);
-
-/** Insert a new input pad for the filter. */
-static inline int ff_append_inpad(AVFilterContext *f,
-                                   AVFilterPad *p)
-{
-    return ff_append_pad(&f->nb_inputs,
-                  &f->input_pads, &f->inputs, p);
-}
-
-/** Insert a new output pad for the filter. */
-static inline int ff_append_outpad(AVFilterContext *f,
-                                    AVFilterPad *p)
-{
-    return ff_append_pad(&f->nb_outputs,
-                  &f->output_pads, &f->outputs, p);
-}
+int ff_append_inpad (AVFilterContext *f, AVFilterPad *p);
+int ff_append_outpad(AVFilterContext *f, AVFilterPad *p);
 
 /**
  * Request an input frame from the filter at the other end of the link.



More information about the ffmpeg-cvslog mailing list