[FFmpeg-devel] [PATCH] Partially revert "swr: add prototypes for resample dsp functions"

James Almer jamrial at gmail.com
Tue Jul 1 23:45:21 CEST 2014


Prototypes are not needed anymore now that the x86 functions don't
include resample_template.c

The DO_RESAMPLE_ONE macro is removed for that same reason as well.

Signed-off-by: James Almer <jamrial at gmail.com>
---
 libswresample/resample_dsp.c      | 29 ++++++++---------------------
 libswresample/resample_template.c | 14 ++++++--------
 2 files changed, 14 insertions(+), 29 deletions(-)

diff --git a/libswresample/resample_dsp.c b/libswresample/resample_dsp.c
index a3b600f..b792d42 100644
--- a/libswresample/resample_dsp.c
+++ b/libswresample/resample_dsp.c
@@ -27,17 +27,6 @@
 
 #include "resample.h"
 
-int swri_resample_common_int16 (ResampleContext *c, int16_t *dst, const int16_t *src, int n, int update_ctx);
-int swri_resample_common_int32 (ResampleContext *c, int32_t *dst, const int32_t *src, int n, int update_ctx);
-int swri_resample_common_float (ResampleContext *c,   float *dst, const   float *src, int n, int update_ctx);
-int swri_resample_common_double(ResampleContext *c,  double *dst, const  double *src, int n, int update_ctx);
-int swri_resample_linear_int16 (ResampleContext *c, int16_t *dst, const int16_t *src, int n, int update_ctx);
-int swri_resample_linear_int32 (ResampleContext *c, int32_t *dst, const int32_t *src, int n, int update_ctx);
-int swri_resample_linear_float (ResampleContext *c,   float *dst, const   float *src, int n, int update_ctx);
-int swri_resample_linear_double(ResampleContext *c,  double *dst, const  double *src, int n, int update_ctx);
-
-#define DO_RESAMPLE_ONE 1
-
 #define TEMPLATE_RESAMPLE_S16
 #include "resample_template.c"
 #undef TEMPLATE_RESAMPLE_S16
@@ -54,8 +43,6 @@ int swri_resample_linear_double(ResampleContext *c,  double *dst, const  double
 #include "resample_template.c"
 #undef TEMPLATE_RESAMPLE_DBL
 
-#undef DO_RESAMPLE_ONE
-
 void swresample_dsp_init(ResampleContext *c)
 {
 #define FNIDX(fmt) (AV_SAMPLE_FMT_##fmt - AV_SAMPLE_FMT_S16P)
@@ -64,15 +51,15 @@ void swresample_dsp_init(ResampleContext *c)
     c->dsp.resample_one[FNIDX(FLTP)] = (resample_one_fn) resample_one_float;
     c->dsp.resample_one[FNIDX(DBLP)] = (resample_one_fn) resample_one_double;
 
-    c->dsp.resample_common[FNIDX(S16P)] = (resample_fn) swri_resample_common_int16;
-    c->dsp.resample_common[FNIDX(S32P)] = (resample_fn) swri_resample_common_int32;
-    c->dsp.resample_common[FNIDX(FLTP)] = (resample_fn) swri_resample_common_float;
-    c->dsp.resample_common[FNIDX(DBLP)] = (resample_fn) swri_resample_common_double;
+    c->dsp.resample_common[FNIDX(S16P)] = (resample_fn) resample_common_int16;
+    c->dsp.resample_common[FNIDX(S32P)] = (resample_fn) resample_common_int32;
+    c->dsp.resample_common[FNIDX(FLTP)] = (resample_fn) resample_common_float;
+    c->dsp.resample_common[FNIDX(DBLP)] = (resample_fn) resample_common_double;
 
-    c->dsp.resample_linear[FNIDX(S16P)] = (resample_fn) swri_resample_linear_int16;
-    c->dsp.resample_linear[FNIDX(S32P)] = (resample_fn) swri_resample_linear_int32;
-    c->dsp.resample_linear[FNIDX(FLTP)] = (resample_fn) swri_resample_linear_float;
-    c->dsp.resample_linear[FNIDX(DBLP)] = (resample_fn) swri_resample_linear_double;
+    c->dsp.resample_linear[FNIDX(S16P)] = (resample_fn) resample_linear_int16;
+    c->dsp.resample_linear[FNIDX(S32P)] = (resample_fn) resample_linear_int32;
+    c->dsp.resample_linear[FNIDX(FLTP)] = (resample_fn) resample_linear_float;
+    c->dsp.resample_linear[FNIDX(DBLP)] = (resample_fn) resample_linear_double;
 
     if (ARCH_X86) swresample_dsp_x86_init(c);
 }
diff --git a/libswresample/resample_template.c b/libswresample/resample_template.c
index 4f1638e..ab546f1 100644
--- a/libswresample/resample_template.c
+++ b/libswresample/resample_template.c
@@ -70,7 +70,6 @@
 
 #endif
 
-#if DO_RESAMPLE_ONE
 static void RENAME(resample_one)(DELEM *dst, const DELEM *src,
                                  int dst_size, int64_t index2, int64_t incr)
 {
@@ -81,11 +80,10 @@ static void RENAME(resample_one)(DELEM *dst, const DELEM *src,
         index2 += incr;
     }
 }
-#endif
 
-int RENAME(swri_resample_common)(ResampleContext *c,
-                                 DELEM *dst, const DELEM *src,
-                                 int n, int update_ctx)
+static int RENAME(resample_common)(ResampleContext *c,
+                                   DELEM *dst, const DELEM *src,
+                                   int n, int update_ctx)
 {
     int dst_index;
     int index= c->index;
@@ -121,9 +119,9 @@ int RENAME(swri_resample_common)(ResampleContext *c,
     return sample_index;
 }
 
-int RENAME(swri_resample_linear)(ResampleContext *c,
-                                 DELEM *dst, const DELEM *src,
-                                 int n, int update_ctx)
+static int RENAME(resample_linear)(ResampleContext *c,
+                                   DELEM *dst, const DELEM *src,
+                                   int n, int update_ctx)
 {
     int dst_index;
     int index= c->index;
-- 
1.8.5.5



More information about the ffmpeg-devel mailing list