<p dir="ltr">Le 2 août 2016 20:05, Hugo Beauzée-Luyssen <hugo@beauzee.fr> a écrit :<br>
><br>
> On 08/02/2016 06:59 PM, Hugo Beauzée-Luyssen wrote:<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>
> Hi,<br>
><br>
> Sorry for the lack of comment... basically the idea is this change <br>
> allows one to see the log on some platform that would swallow <br>
> stdout/stderr, making debugging easier.</p>
<p dir="ltr">Meh. On a platform does not have standard error and output streams, why would one use the CLI executable in the first place? IIRC, we do not even have the normal executable on such platform.</p>
<p dir="ltr">> It seems weird that this wasn't done before, yet I can't see a reason <br>
> for it, hence the patch.</p>
<p dir="ltr">Very much on purpose, AFAICT.<br>
><br>
> Regards,<br>
> _______________________________________________<br>
> vlc-devel mailing list<br>
> To unsubscribe or modify your subscription options:<br>
> https://mailman.videolan.org/listinfo/vlc-devel<br></p>