[FFmpeg-devel] [PATCH] lavfi/af_aformat: remove support for comma-separated lists

[James Almer]

On 3/2/2022 1:04 PM, Nicolas George wrote:
> It has been deprecated for nine years.

We need to make sure to add FF_API_ defines when we introduce this kind 
of deprecation in the future, as it forces people to at least look at 
them at every major bump. Nine years is definitely overkill.

> 
> Signed-off-by: Nicolas George <george at nsup.org>
> ---
>   libavfilter/af_aformat.c | 11 ++---------
>   1 file changed, 2 insertions(+), 9 deletions(-)
> 
> 
> I want to share the delimiter-seeking function with vf_format and add
> support for '/'.
> 
> 
> diff --git a/libavfilter/af_aformat.c b/libavfilter/af_aformat.c
> index ed3c75311a..d2599431dc 100644
> --- a/libavfilter/af_aformat.c
> +++ b/libavfilter/af_aformat.c
> @@ -62,19 +62,12 @@ AVFILTER_DEFINE_CLASS(aformat);
>   
>   #define PARSE_FORMATS(str, type, list, add_to_list, get_fmt, none, desc)    \
>   do {                                                                        \
> -    char *next, *cur = str, sep;                                            \
> +    char *next, *cur = str;                                                 \
>       int ret;                                                                \
>                                                                               \
> -    if (str && strchr(str, ',')) {                                          \
> -        av_log(ctx, AV_LOG_WARNING, "This syntax is deprecated, use '|' to "\
> -               "separate %s.\n", desc);                                     \
> -        sep = ',';                                                          \
> -    } else                                                                  \
> -        sep = '|';                                                          \
> -                                                                            \
>       while (cur) {                                                           \
>           type fmt;                                                           \
> -        next = strchr(cur, sep);                                            \
> +        next = strchr(cur, '|');                                            \
>           if (next)                                                           \
>               *next++ = 0;                                                    \
>                                                                               \

LGTM.