[FFmpeg-cvslog] avfilter/formats: Avoid code duplication when merging samplerates

Andreas Rheinhardt git at videolan.org
Fri Aug 21 01:30:43 EEST 2020


ffmpeg | branch: master | Andreas Rheinhardt <andreas.rheinhardt at gmail.com> | Fri Aug 14 00:18:09 2020 +0200| [363f93460fbce0a2ea073f7f64edc597405982a1] | committer: Andreas Rheinhardt

avfilter/formats: Avoid code duplication when merging samplerates

Right now, ff_merge_samplerates() contains three instances of the
MERGE_REF() macro, a macro which reallocates an array, updates some
pointers in a loop and frees several buffers. This commit makes it
possible to contain only one instance of said macro in the function,
thereby reducing codesize.

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at gmail.com>

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

 libavfilter/formats.c | 31 +++++++++++++++++--------------
 1 file changed, 17 insertions(+), 14 deletions(-)

diff --git a/libavfilter/formats.c b/libavfilter/formats.c
index 28433dcd52..2a40762897 100644
--- a/libavfilter/formats.c
+++ b/libavfilter/formats.c
@@ -56,11 +56,21 @@ do {                                                                       \
 
 /**
  * Add all formats common to a and b to a, add b's refs to a and destroy b.
+ * If empty_allowed is set and one of a,b->nb is zero, the lists are
+ * merged; otherwise, it is treated as error.
  */
-#define MERGE_FORMATS(a, b, fmts, nb, type, fail_statement)                \
+#define MERGE_FORMATS(a, b, fmts, nb, type, fail_statement, empty_allowed) \
 do {                                                                            \
-    int i, j, k = 0;                                                       \
+    int i, j, k = 0, skip = 0;                                             \
                                                                                 \
+    if (empty_allowed) {                                                   \
+        if (!a->nb || !b->nb) {                                            \
+            if (!a->nb)                                                    \
+                FFSWAP(type *, a, b);                                      \
+            skip = 1;                                                      \
+        }                                                                  \
+    }                                                                      \
+    if (!skip) {                                                           \
         for (i = 0; i < a->nb; i++)                                             \
             for (j = 0; j < b->nb; j++)                                         \
                 if (a->fmts[i] == b->fmts[j]) {                                 \
@@ -72,7 +82,8 @@ do {
     if (!k)                                                                \
         { fail_statement }                                                 \
     a->nb = k;                                                             \
-                                                                                \
+    }                                                                      \
+                                                                           \
     MERGE_REF(a, b, fmts, type, fail_statement);                           \
 } while (0)
 
@@ -110,7 +121,7 @@ AVFilterFormats *ff_merge_formats(AVFilterFormats *a, AVFilterFormats *b,
     if (alpha2 > alpha1 || chroma2 > chroma1)
         return NULL;
 
-    MERGE_FORMATS(a, b, formats, nb_formats, AVFilterFormats, return NULL;);
+    MERGE_FORMATS(a, b, formats, nb_formats, AVFilterFormats, return NULL;, 0);
 
     return a;
 }
@@ -120,16 +131,8 @@ AVFilterFormats *ff_merge_samplerates(AVFilterFormats *a,
 {
     if (a == b) return a;
 
-    if (a->nb_formats && b->nb_formats) {
-        MERGE_FORMATS(a, b, formats, nb_formats, AVFilterFormats, return NULL;);
-        return a;
-    } else if (a->nb_formats) {
-        MERGE_REF(a, b, formats, AVFilterFormats, return NULL;);
-        return a;
-    } else {
-        MERGE_REF(b, a, formats, AVFilterFormats, return NULL;);
-        return b;
-    }
+    MERGE_FORMATS(a, b, formats, nb_formats, AVFilterFormats, return NULL;, 1);
+    return a;
 }
 
 AVFilterChannelLayouts *ff_merge_channel_layouts(AVFilterChannelLayouts *a,



More information about the ffmpeg-cvslog mailing list