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 <[email protected]>
---
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.
_______________________________________________
ffmpeg-devel mailing list
[email protected]
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
[email protected] with subject "unsubscribe".
