[FFmpeg-devel] [PATCH] cmdutils: remove error message from opt_default() in case of missing option

[Clément Bœsch]

On Thu, Apr 04, 2013 at 10:23:32PM +0200, Stefano Sabatini wrote:
> A similar error is generated in the calling function, avoid duplication.
> ---
>  cmdutils.c |    2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/cmdutils.c b/cmdutils.c
> index 51ec634..831c528 100644
> --- a/cmdutils.c
> +++ b/cmdutils.c
> @@ -547,8 +547,6 @@ int opt_default(void *optctx, const char *opt, const char *arg)
>  
>      if (consumed)
>          return 0;
> -    av_log(NULL, AV_LOG_ERROR, "Could not find option '%s' in any of the FFmpeg subsystems "
> -           "(codec, format, scaler, resampler contexts)\n", opt);
>      return AVERROR_OPTION_NOT_FOUND;
>  }

LGTM, thanks, the -nostats false positive message was annoying :)

-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 490 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20130404/1f129c4b/attachment.asc>