[MPlayer-dev-eng] [PATCH] quit return code

[Sascha Sommer]

On Monday 13 September 2004 22:55, Aurelien Jacobs wrote:
> On Sat, 11 Sep 2004 18:58:57 +0900
>
> Attila Kinali <attila at kinali.ch> wrote:
> > On Fri, Aug 27, 2004 at 01:36:01PM +0200, Aurelien Jacobs wrote:
> > > This patch modify the "quit" slave command so that we can add an
> > > optionnal integer parameter to it, that will be returned by mplayer.
> > > If you think this patch is OK, I will improve doc accordingly.
> >
> > +        exit_player_with_rc(MSGTR_Exit_quit,
> > +                            cmd->nargs > 0 ? cmd->args[0].v.i : 0);
> >
> > What happends if i type "quit blah" ?
>
> It didn't analysed the whole code but I guess the option parser will
> only accept integer arguments with the following line ?
>   { MP_CMD_QUIT, "quit", 0, { {MP_CMD_ARG_INT,{0}}, {-1,{0}} } },
>
> In practice, "quit blah" simply exit with return code 0, which is IMHO
> the right behavior.
>

Applied. Please send a patch to update the documentation.

Sascha