[FFmpeg-devel] [PATCH 2/2] fftools/ffmpeg_opt: remove bogus warning of multiple -af and -vf usage

Paul B Mahol onemda at gmail.com
Sun Mar 1 11:10:01 EET 2020


lgtm

On 2/29/20, Marton Balint <cus at passwd.hu> wrote:
> This is redundant after the last patch and also fixes ticket #7712.
>
> Signed-off-by: Marton Balint <cus at passwd.hu>
> ---
>  fftools/ffmpeg_opt.c | 4 ----
>  1 file changed, 4 deletions(-)
>
> diff --git a/fftools/ffmpeg_opt.c b/fftools/ffmpeg_opt.c
> index 3d6fafe073..df6928ee10 100644
> --- a/fftools/ffmpeg_opt.c
> +++ b/fftools/ffmpeg_opt.c
> @@ -1746,8 +1746,6 @@ static OutputStream *new_video_stream(OptionsContext
> *o, AVFormatContext *oc, in
>
>      MATCH_PER_STREAM_OPT(filter_scripts, str, ost->filters_script, oc, st);
>      MATCH_PER_STREAM_OPT(filters,        str, ost->filters,        oc, st);
> -    if (o->nb_filters > 1)
> -        av_log(NULL, AV_LOG_ERROR, "Only '-vf %s' read, ignoring remaining
> -vf options: Use ',' to separate filters\n", ost->filters);
>
>      if (!ost->stream_copy) {
>          const char *p = NULL;
> @@ -1929,8 +1927,6 @@ static OutputStream *new_audio_stream(OptionsContext
> *o, AVFormatContext *oc, in
>
>      MATCH_PER_STREAM_OPT(filter_scripts, str, ost->filters_script, oc, st);
>      MATCH_PER_STREAM_OPT(filters,        str, ost->filters,        oc, st);
> -    if (o->nb_filters > 1)
> -        av_log(NULL, AV_LOG_ERROR, "Only '-af %s' read, ignoring remaining
> -af options: Use ',' to separate filters\n", ost->filters);
>
>      if (!ost->stream_copy) {
>          char *sample_fmt = NULL;
> --
> 2.16.4
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".


More information about the ffmpeg-devel mailing list