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

[Aurelien Jacobs]

On Wed, 15 Sep 2004 11:30:28 +0000
Sascha Sommer <saschasommer at freenet.de> wrote:

> 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.

Thanks.
Here is the documentation update.

Aurel
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: MPlayer-quit-doc.diff
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20040916/fd6dae00/attachment.asc>