diff options
author | Stefano Sabatini <stefano.sabatini-lala@poste.it> | 2011-09-13 01:46:29 +0200 |
---|---|---|
committer | Stefano Sabatini <stefasab@gmail.com> | 2011-09-16 00:42:47 +0200 |
commit | 9899037dc6b246f2eeb7e0255c316749372fe1ae (patch) | |
tree | f3aa2650d5fb63f567fd31fe1edabda917480203 | |
parent | 98b906e1af789e9bc9930c2d2bfa3d0798405e21 (diff) | |
download | ffmpeg-9899037dc6b246f2eeb7e0255c316749372fe1ae.tar.gz |
lavfi: rename avfilter_all_* function names to avfilter_make_all_*
A function name should tell what the function does rather than what
the function returns. This also avoids possible conflicts
(e.g. between a function and a public list of all supported formats),
and clarifies the code.
Breaks audio API/ABI, add a backward compatibility layer for video
API/ABI.
-rw-r--r-- | doc/APIchanges | 8 | ||||
-rw-r--r-- | libavfilter/af_aformat.c | 6 | ||||
-rw-r--r-- | libavfilter/af_aresample.c | 4 | ||||
-rw-r--r-- | libavfilter/avfilter.h | 19 | ||||
-rw-r--r-- | libavfilter/defaults.c | 8 | ||||
-rw-r--r-- | libavfilter/formats.c | 11 |
6 files changed, 41 insertions, 15 deletions
diff --git a/doc/APIchanges b/doc/APIchanges index fe02eb2c0b..3e126516c2 100644 --- a/doc/APIchanges +++ b/doc/APIchanges @@ -13,6 +13,14 @@ libavutil: 2011-04-18 API changes, most recent first: +2011-09-16 - xxxxxxx - lavfi 2.41.0 + Rename avfilter_all_* function names to avfilter_make_all_*. + + In particular, apply the renames: + avfilter_all_formats -> avfilter_make_all_formats + avfilter_all_channel_layouts -> avfilter_make_all_channel_layouts + avfilter_all_packing_formats -> avfilter_make_all_packing_formats + 2011-09-12 - xxxxxxx - lavfi 2.40.0 Change AVFilterBufferRefAudioProps.sample_rate type from uint32_t to int. diff --git a/libavfilter/af_aformat.c b/libavfilter/af_aformat.c index 0eb7dfa4e2..a7aa45f0b1 100644 --- a/libavfilter/af_aformat.c +++ b/libavfilter/af_aformat.c @@ -63,9 +63,9 @@ static av_cold int init(AVFilterContext *ctx, const char *args, void *opaque) if (*args) \ args++; - ADD_FORMATS(avfilter_all_formats(AVMEDIA_TYPE_AUDIO), sample_format, int, formats); - ADD_FORMATS(avfilter_all_channel_layouts(), channel_layout, int64_t, chlayouts); - ADD_FORMATS(avfilter_all_packing_formats(), packing_format, int, packing); + ADD_FORMATS(avfilter_make_all_formats(AVMEDIA_TYPE_AUDIO), sample_format, int, formats); + ADD_FORMATS(avfilter_make_all_channel_layouts(), channel_layout, int64_t, chlayouts); + ADD_FORMATS(avfilter_make_all_packing_formats(), packing_format, int, packing); return 0; diff --git a/libavfilter/af_aresample.c b/libavfilter/af_aresample.c index fa26788254..04188c854d 100644 --- a/libavfilter/af_aresample.c +++ b/libavfilter/af_aresample.c @@ -104,12 +104,12 @@ static int query_formats(AVFilterContext *ctx) return AVERROR(ENOMEM); avfilter_set_common_sample_formats(ctx, formats); - formats = avfilter_all_channel_layouts(); + formats = avfilter_make_all_channel_layouts(); if (!formats) return AVERROR(ENOMEM); avfilter_set_common_channel_layouts(ctx, formats); - formats = avfilter_all_packing_formats(); + formats = avfilter_make_all_packing_formats(); if (!formats) return AVERROR(ENOMEM); avfilter_set_common_packing_formats(ctx, formats); diff --git a/libavfilter/avfilter.h b/libavfilter/avfilter.h index 90bb4d17fb..a8426fea53 100644 --- a/libavfilter/avfilter.h +++ b/libavfilter/avfilter.h @@ -29,7 +29,7 @@ #include "libavutil/rational.h" #define LIBAVFILTER_VERSION_MAJOR 2 -#define LIBAVFILTER_VERSION_MINOR 40 +#define LIBAVFILTER_VERSION_MINOR 41 #define LIBAVFILTER_VERSION_MICRO 0 #define LIBAVFILTER_VERSION_INT AV_VERSION_INT(LIBAVFILTER_VERSION_MAJOR, \ @@ -43,6 +43,9 @@ #ifndef FF_API_OLD_VSINK_API #define FF_API_OLD_VSINK_API (LIBAVUTIL_VERSION_MAJOR < 3) #endif +#ifndef FF_API_OLD_ALL_FORMATS_API +#define FF_API_OLD_ALL_FORMATS_API (LIBAVUTIL_VERSION_MAJOR < 3) +#endif #include <stddef.h> @@ -258,20 +261,28 @@ AVFilterFormats *avfilter_make_format64_list(const int64_t *fmts); */ int avfilter_add_format(AVFilterFormats **avff, int64_t fmt); +#if FF_API_OLD_ALL_FORMATS_API /** - * Return a list of all formats supported by FFmpeg for the given media type. + * @deprecated Use avfilter_make_all_formats() instead. */ +attribute_deprecated AVFilterFormats *avfilter_all_formats(enum AVMediaType type); +#endif + +/** + * Return a list of all formats supported by FFmpeg for the given media type. + */ +AVFilterFormats *avfilter_make_all_formats(enum AVMediaType type); /** * Return a list of all channel layouts supported by FFmpeg. */ -AVFilterFormats *avfilter_all_channel_layouts(void); +AVFilterFormats *avfilter_make_all_channel_layouts(void); /** * Return a list of all audio packing formats. */ -AVFilterFormats *avfilter_all_packing_formats(void); +AVFilterFormats *avfilter_make_all_packing_formats(void); /** * Return a format list which contains the intersection of the formats of diff --git a/libavfilter/defaults.c b/libavfilter/defaults.c index c231f75d85..79fb38d70f 100644 --- a/libavfilter/defaults.c +++ b/libavfilter/defaults.c @@ -222,10 +222,10 @@ void avfilter_set_common_packing_formats(AVFilterContext *ctx, AVFilterFormats * int avfilter_default_query_formats(AVFilterContext *ctx) { - avfilter_set_common_pixel_formats(ctx, avfilter_all_formats(AVMEDIA_TYPE_VIDEO)); - avfilter_set_common_sample_formats(ctx, avfilter_all_formats(AVMEDIA_TYPE_AUDIO)); - avfilter_set_common_channel_layouts(ctx, avfilter_all_channel_layouts()); - avfilter_set_common_packing_formats(ctx, avfilter_all_packing_formats()); + avfilter_set_common_pixel_formats(ctx, avfilter_make_all_formats(AVMEDIA_TYPE_VIDEO)); + avfilter_set_common_sample_formats(ctx, avfilter_make_all_formats(AVMEDIA_TYPE_AUDIO)); + avfilter_set_common_channel_layouts(ctx, avfilter_make_all_channel_layouts()); + avfilter_set_common_packing_formats(ctx, avfilter_make_all_packing_formats()); return 0; } diff --git a/libavfilter/formats.c b/libavfilter/formats.c index ef58665a17..0efa1aad77 100644 --- a/libavfilter/formats.c +++ b/libavfilter/formats.c @@ -138,8 +138,15 @@ int avfilter_add_format(AVFilterFormats **avff, int64_t fmt) return 0; } +#if FF_API_OLD_ALL_FORMATS_API AVFilterFormats *avfilter_all_formats(enum AVMediaType type) { + return avfilter_make_all_formats(type); +} +#endif + +AVFilterFormats *avfilter_make_all_formats(enum AVMediaType type) +{ AVFilterFormats *ret = NULL; int fmt; int num_formats = type == AVMEDIA_TYPE_VIDEO ? PIX_FMT_NB : @@ -153,7 +160,7 @@ AVFilterFormats *avfilter_all_formats(enum AVMediaType type) return ret; } -AVFilterFormats *avfilter_all_channel_layouts(void) +AVFilterFormats *avfilter_make_all_channel_layouts(void) { static int64_t chlayouts[] = { AV_CH_LAYOUT_MONO, @@ -174,7 +181,7 @@ AVFilterFormats *avfilter_all_channel_layouts(void) return avfilter_make_format64_list(chlayouts); } -AVFilterFormats *avfilter_all_packing_formats(void) +AVFilterFormats *avfilter_make_all_packing_formats(void) { static int packing[] = { AVFILTER_PACKED, |