<p dir="ltr">This breaks the established defacto standard for CLI syntax errors on Unix. (IIRC, distcheck enforces it, but not sure.)</p>
<p dir="ltr">Also, I don't think CLI user cares about the extra infos provided by vlc_Log.</p>
<p dir="ltr">Le 2 août 2016 19:59, Hugo Beauzée-Luyssen <hugo@beauzee.fr> a écrit :<br>
><br>
> ---<br>
> src/config/cmdline.c | 14 +++++++-------<br>
> 1 file changed, 7 insertions(+), 7 deletions(-)<br>
><br>
> diff --git a/src/config/cmdline.c b/src/config/cmdline.c<br>
> index 12e92ba..12045e9 100644<br>
> --- a/src/config/cmdline.c<br>
> +++ b/src/config/cmdline.c<br>
> @@ -225,8 +225,8 @@ int config_LoadCmdLine( vlc_object_t *p_this, int i_argc,<br>
>                  /* Check if the option is deprecated */<br>
>                  if( p_conf->b_removed )<br>
>                  {<br>
> -                    fprintf(stderr,<br>
> -                            "Warning: option --%s no longer exists.\n",<br>
> +                    msg_Warn(p_this,<br>
> +                            "Warning: option --%s no longer exists.",<br>
>                              psz_name);<br>
>                      continue;<br>
>                  }<br>
> @@ -290,17 +290,17 @@ int config_LoadCmdLine( vlc_object_t *p_this, int i_argc,<br>
>          /* Internal error: unknown option */<br>
>          if( !b_ignore_errors )<br>
>          {<br>
> -            fputs( "vlc: unknown option"<br>
> -                     " or missing mandatory argument ", stderr );<br>
>              if( state.opt )<br>
>              {<br>
> -                fprintf( stderr, "`-%c'\n", state.opt );<br>
> +                msg_Err( p_this, "vlc: unknown option"<br>
> +                         " or missing mandatory argument `-%c'", state.opt );<br>
>              }<br>
>              else<br>
>              {<br>
> -                fprintf( stderr, "`%s'\n", ppsz_argv[state.ind-1] );<br>
> +                msg_Err( p_this, "vlc: unknown option"<br>
> +                         " or missing mandatory argument `-%s'", ppsz_argv[state.ind-1] );<br>
>              }<br>
> -            fputs( "Try `vlc --help' for more information.\n", stderr );<br>
> +            msg_Err( p_this, "Try `vlc --help' for more information." );<br>
>              goto out;<br>
>          }<br>
>      }<br>
> -- <br>
> 2.8.1<br>
><br>
> _______________________________________________<br>
> vlc-devel mailing list<br>
> To unsubscribe or modify your subscription options:<br>
> https://mailman.videolan.org/listinfo/vlc-devel<br></p>