From patchwork Mon Jan 14 12:39:11 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Carl Eugen Hoyos X-Patchwork-Id: 11738 Return-Path: X-Original-To: patchwork@ffaux-bg.ffmpeg.org Delivered-To: patchwork@ffaux-bg.ffmpeg.org Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org [79.124.17.100]) by ffaux.localdomain (Postfix) with ESMTP id 199FF44D19B for ; Mon, 14 Jan 2019 14:39:18 +0200 (EET) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 3989D68A89F; Mon, 14 Jan 2019 14:39:06 +0200 (EET) X-Original-To: ffmpeg-devel@ffmpeg.org Delivered-To: ffmpeg-devel@ffmpeg.org Received: from mail-io1-f66.google.com (mail-io1-f66.google.com [209.85.166.66]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 651D8689B79 for ; Mon, 14 Jan 2019 14:38:59 +0200 (EET) Received: by mail-io1-f66.google.com with SMTP id v10so17428809ios.13 for ; Mon, 14 Jan 2019 04:39:13 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:from:date:message-id:subject:to; bh=noEe4nTmxwvGEBrNxMEnzXSYUA5IxiDuQkZiBHgyepc=; b=nsfmzc0qFTiiekV8eXBRaAZ2R+ww/f+OEhgRPo5F/Zy3Urg6+lsOMf7V/Jt2/NFI0l 66IzWmRqSg9iAPw2NSd3plGthOLkZgoVcrWSZxoJdV93xmGQZvPFTwzSNfqva2FDPWf+ 5c2SR3aKLsv+VRWZXkv6TqLkJcA1SjuL0rzTu8mJQjAGzUOvs38JyO180sQCPOG+nIHL 4PZmmImzIx30AuZYk1I13TlxAkiYExq9EFYtzWfY69ADWk2vMI2dSzTMlcqpA8tF874r Ms/h3oT7X+uM2fPhYtpbBElwgJ6RAkO1bJOgf5SLXXk3fhW2shaJ9jVERHTeZhfcsY8h I/9g== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:from:date:message-id:subject:to; bh=noEe4nTmxwvGEBrNxMEnzXSYUA5IxiDuQkZiBHgyepc=; b=joz32yBwdI1GN/0kbCmRLhnMW2f0yIl8BO7T+eF+8tyN7nxg3k/BvRYkmwAVdZ9XDI evDND7tR2SO9+WcRFZnvprLCHfWrn1H89JIV7rtb6l3Ds0+RCeJTD4/RIPz5YQMpFv47 RzYA2FqocSdvXesVIouYtD6w2p4/W0579Rf8rfHjQyiWpMWVvjHoI70NgsZ050QvI9Ay mLc8waMILt7ZOgtFcfaOa4uVz8n00KbiQ/DX02DgtRYlVK5CSmly2mC5/aW4q3L8lU0F njr8qGwl4ASMwmsmJMmG0faOFnkWdlkr7HCbO8aFAPCB33B9rVWUWrhEBR5TsI+g6pRO 6Ytg== X-Gm-Message-State: AJcUukeP+jeKzUm9UlWLnQ+Ml4jsTa3zEc44f1J442LKpe4VCit026+U eloyxreqw7m6OeYCHEaAWv6llOiQd3/m0wluk6ZQOw== X-Google-Smtp-Source: ALg8bN64/Am3JwsHJcQKKFJ8JnPRwbW0I+pm4jlTqScqnF1fsEaViKEsjwkFDWXPBEA0EVI3EgYymvm+OFmIhgJ9sJc= X-Received: by 2002:a5d:9405:: with SMTP id v5mr16783679ion.250.1547469551930; Mon, 14 Jan 2019 04:39:11 -0800 (PST) MIME-Version: 1.0 Received: by 2002:a02:5f11:0:0:0:0:0 with HTTP; Mon, 14 Jan 2019 04:39:11 -0800 (PST) From: Carl Eugen Hoyos Date: Mon, 14 Jan 2019 13:39:11 +0100 Message-ID: To: FFmpeg development discussions and patches Subject: [FFmpeg-devel] [PATCH]ffmpeg_opt: Warn if more than one -vf/-af option was specified X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.20 Precedence: list List-Id: FFmpeg development discussions and patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: FFmpeg development discussions and patches Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Hi! This user mistake affects many people, attached patch fixes ticket #4184. Please comment, Carl Eugen From cb10736ba113a7791816a4db2478eb25f148100f Mon Sep 17 00:00:00 2001 From: Carl Eugen Hoyos Date: Mon, 14 Jan 2019 13:34:22 +0100 Subject: [PATCH] ffmpeg_opt: Print a warning if more than one -vf/-af option was specified. Fixes ticket #4184. --- fftools/ffmpeg_opt.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/fftools/ffmpeg_opt.c b/fftools/ffmpeg_opt.c index a99e762..b5dfd71 100644 --- a/fftools/ffmpeg_opt.c +++ b/fftools/ffmpeg_opt.c @@ -1681,6 +1681,8 @@ 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\n", ost->filters); if (!ost->stream_copy) { const char *p = NULL; @@ -1862,6 +1864,8 @@ 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\n", ost->filters); if (!ost->stream_copy) { char *sample_fmt = NULL; -- 1.7.10.4