aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAndreas Rheinhardt <andreas.rheinhardt@gmail.com>2020-08-08 00:37:46 +0200
committerAndreas Rheinhardt <andreas.rheinhardt@gmail.com>2020-08-12 21:10:59 +0200
commitae5026c90502fad71338b81badd227db00b462cf (patch)
treea0c6c1053c879785d4f125c162eb82c3268dfa93
parent2e0cf4de01ed062fc7862c821c21bb578dd4c5f8 (diff)
downloadffmpeg-ae5026c90502fad71338b81badd227db00b462cf.tar.gz
avfilter/formats: Schedule avfilter_make_format64_list() for removal
Despite its name, this function is not part of the public API, as formats.h, the header containing its declaration, is a private header. The formats API was once public API, but that changed long ago (b74a1da49db5ebed51aceae6cacc2329288a92c1, the commit scheduling it to become private, is from 2012). That avfilter_make_format64_list() was forgotten is probably a result of the confusion resulting from the libav-ffmpeg split. Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
-rw-r--r--libavfilter/aeval.c2
-rw-r--r--libavfilter/af_aresample.c2
-rw-r--r--libavfilter/asrc_afirsrc.c2
-rw-r--r--libavfilter/asrc_anoisesrc.c2
-rw-r--r--libavfilter/asrc_anullsrc.c3
-rw-r--r--libavfilter/asrc_hilbert.c2
-rw-r--r--libavfilter/asrc_sinc.c2
-rw-r--r--libavfilter/asrc_sine.c2
-rw-r--r--libavfilter/avf_showcqt.c2
-rw-r--r--libavfilter/formats.c9
-rw-r--r--libavfilter/formats.h6
-rw-r--r--libavfilter/src_movie.c2
12 files changed, 23 insertions, 13 deletions
diff --git a/libavfilter/aeval.c b/libavfilter/aeval.c
index 855dc0009d..c7a8cd6550 100644
--- a/libavfilter/aeval.c
+++ b/libavfilter/aeval.c
@@ -258,7 +258,7 @@ static int query_formats(AVFilterContext *ctx)
if (ret < 0)
return ret;
- layouts = avfilter_make_format64_list(chlayouts);
+ layouts = ff_make_format64_list(chlayouts);
if (!layouts)
return AVERROR(ENOMEM);
ret = ff_set_common_channel_layouts(ctx, layouts);
diff --git a/libavfilter/af_aresample.c b/libavfilter/af_aresample.c
index fb10bd1c33..b7b1b5ba3c 100644
--- a/libavfilter/af_aresample.c
+++ b/libavfilter/af_aresample.c
@@ -126,7 +126,7 @@ static int query_formats(AVFilterContext *ctx)
if(out_layout) {
int64_t layout_list[] = { out_layout, -1 };
- out_layouts = avfilter_make_format64_list(layout_list);
+ out_layouts = ff_make_format64_list(layout_list);
} else
out_layouts = ff_all_channel_counts();
diff --git a/libavfilter/asrc_afirsrc.c b/libavfilter/asrc_afirsrc.c
index b90ffad57f..d8ba8f6a54 100644
--- a/libavfilter/asrc_afirsrc.c
+++ b/libavfilter/asrc_afirsrc.c
@@ -145,7 +145,7 @@ static av_cold int query_formats(AVFilterContext *ctx)
if (ret < 0)
return ret;
- layouts = avfilter_make_format64_list(chlayouts);
+ layouts = ff_make_format64_list(chlayouts);
if (!layouts)
return AVERROR(ENOMEM);
ret = ff_set_common_channel_layouts(ctx, layouts);
diff --git a/libavfilter/asrc_anoisesrc.c b/libavfilter/asrc_anoisesrc.c
index 7aa878707d..40ee077001 100644
--- a/libavfilter/asrc_anoisesrc.c
+++ b/libavfilter/asrc_anoisesrc.c
@@ -101,7 +101,7 @@ static av_cold int query_formats(AVFilterContext *ctx)
if (ret < 0)
return ret;
- layouts = avfilter_make_format64_list(chlayouts);
+ layouts = ff_make_format64_list(chlayouts);
if (!layouts)
return AVERROR(ENOMEM);
ret = ff_set_common_channel_layouts(ctx, layouts);
diff --git a/libavfilter/asrc_anullsrc.c b/libavfilter/asrc_anullsrc.c
index 52db61685d..7093f08275 100644
--- a/libavfilter/asrc_anullsrc.c
+++ b/libavfilter/asrc_anullsrc.c
@@ -83,11 +83,10 @@ static int query_formats(AVFilterContext *ctx)
int ret;
if ((ret = ff_set_common_formats (ctx, ff_all_formats (AVMEDIA_TYPE_AUDIO))) < 0 ||
- (ret = ff_set_common_channel_layouts (ctx, avfilter_make_format64_list (chlayouts ))) < 0 ||
(ret = ff_set_common_samplerates (ctx, ff_make_format_list (sample_rates ))) < 0)
return ret;
- return 0;
+ return ff_set_common_channel_layouts(ctx, ff_make_format64_list(chlayouts));
}
static int config_props(AVFilterLink *outlink)
diff --git a/libavfilter/asrc_hilbert.c b/libavfilter/asrc_hilbert.c
index a51c676c6f..640dfe46f9 100644
--- a/libavfilter/asrc_hilbert.c
+++ b/libavfilter/asrc_hilbert.c
@@ -113,7 +113,7 @@ static av_cold int query_formats(AVFilterContext *ctx)
if (ret < 0)
return ret;
- layouts = avfilter_make_format64_list(chlayouts);
+ layouts = ff_make_format64_list(chlayouts);
if (!layouts)
return AVERROR(ENOMEM);
ret = ff_set_common_channel_layouts(ctx, layouts);
diff --git a/libavfilter/asrc_sinc.c b/libavfilter/asrc_sinc.c
index 24aeab6e7b..c3ecb864d1 100644
--- a/libavfilter/asrc_sinc.c
+++ b/libavfilter/asrc_sinc.c
@@ -84,7 +84,7 @@ static int query_formats(AVFilterContext *ctx)
if (ret < 0)
return ret;
- layouts = avfilter_make_format64_list(chlayouts);
+ layouts = ff_make_format64_list(chlayouts);
if (!layouts)
return AVERROR(ENOMEM);
ret = ff_set_common_channel_layouts(ctx, layouts);
diff --git a/libavfilter/asrc_sine.c b/libavfilter/asrc_sine.c
index 3a87210b4b..6fe080efb6 100644
--- a/libavfilter/asrc_sine.c
+++ b/libavfilter/asrc_sine.c
@@ -194,7 +194,7 @@ static av_cold int query_formats(AVFilterContext *ctx)
if (ret < 0)
return ret;
- layouts = avfilter_make_format64_list(chlayouts);
+ layouts = ff_make_format64_list(chlayouts);
if (!layouts)
return AVERROR(ENOMEM);
ret = ff_set_common_channel_layouts(ctx, layouts);
diff --git a/libavfilter/avf_showcqt.c b/libavfilter/avf_showcqt.c
index 990db3143b..95efe72628 100644
--- a/libavfilter/avf_showcqt.c
+++ b/libavfilter/avf_showcqt.c
@@ -1334,7 +1334,7 @@ static int query_formats(AVFilterContext *ctx)
if ((ret = ff_formats_ref(formats, &inlink->out_formats)) < 0)
return ret;
- layouts = avfilter_make_format64_list(channel_layouts);
+ layouts = ff_make_format64_list(channel_layouts);
if ((ret = ff_channel_layouts_ref(layouts, &inlink->out_channel_layouts)) < 0)
return ret;
diff --git a/libavfilter/formats.c b/libavfilter/formats.c
index bcc539bb38..48b27d0c53 100644
--- a/libavfilter/formats.c
+++ b/libavfilter/formats.c
@@ -289,7 +289,7 @@ AVFilterFormats *ff_make_format_list(const int *fmts)
return formats;
}
-AVFilterChannelLayouts *avfilter_make_format64_list(const int64_t *fmts)
+AVFilterChannelLayouts *ff_make_format64_list(const int64_t *fmts)
{
MAKE_FORMAT_LIST(AVFilterChannelLayouts,
channel_layouts, nb_channel_layouts);
@@ -300,6 +300,13 @@ AVFilterChannelLayouts *avfilter_make_format64_list(const int64_t *fmts)
return formats;
}
+#if LIBAVFILTER_VERSION_MAJOR < 8
+AVFilterChannelLayouts *avfilter_make_format64_list(const int64_t *fmts)
+{
+ return ff_make_format64_list(fmts);
+}
+#endif
+
#define ADD_FORMAT(f, fmt, unref_fn, type, list, nb) \
do { \
type *fmts; \
diff --git a/libavfilter/formats.h b/libavfilter/formats.h
index 708bee3ee8..ffe7a12d53 100644
--- a/libavfilter/formats.h
+++ b/libavfilter/formats.h
@@ -20,6 +20,7 @@
#define AVFILTER_FORMATS_H
#include "avfilter.h"
+#include "version.h"
/**
* A list of supported formats for one end of a filter link. This is used
@@ -139,8 +140,11 @@ av_warn_unused_result
AVFilterChannelLayouts *ff_all_channel_counts(void);
av_warn_unused_result
-AVFilterChannelLayouts *avfilter_make_format64_list(const int64_t *fmts);
+AVFilterChannelLayouts *ff_make_format64_list(const int64_t *fmts);
+#if LIBAVFILTER_VERSION_MAJOR < 8
+AVFilterChannelLayouts *avfilter_make_format64_list(const int64_t *fmts);
+#endif
/**
* A helper for query_formats() which sets all links to the same list of channel
diff --git a/libavfilter/src_movie.c b/libavfilter/src_movie.c
index 476b8d3d96..64d82e9df1 100644
--- a/libavfilter/src_movie.c
+++ b/libavfilter/src_movie.c
@@ -376,7 +376,7 @@ static int movie_query_formats(AVFilterContext *ctx)
if ((ret = ff_formats_ref(ff_make_format_list(list), &outlink->in_samplerates)) < 0)
return ret;
list64[0] = c->channel_layout;
- if ((ret = ff_channel_layouts_ref(avfilter_make_format64_list(list64),
+ if ((ret = ff_channel_layouts_ref(ff_make_format64_list(list64),
&outlink->in_channel_layouts)) < 0)
return ret;
break;