On 2009/07/07 16:56, Jeffrey Middleton <jefr...@gmail.com> wrote:
> options: support short options; add other options
> 
> In order to support short options, the option parser has been
> replaced. The parser itself comes largely from ncmpc (under the
> GPL); it was largely written by Kalle Wallin.

I don't think that code was particularly good.  That's why I removed
it from ncmpc in favor of GLib's option parser.  mpc should not use
GLib, but why not use getopt() / getopt_long()?

> New options have been
> added, as well as the short form of existing ones. The options are
> now: -v/--verbose, -q/--quiet, -h/--host, -p/--port, -f/--format

Is "-h" a good shortcut for "--host"?  Most programs map "-h" to
"--help".  mpc does not have a "--help" option, but that's another
problem..

> fprintf(stderr,"verbose!\n");

You forgot to remove the debug code..

Max

------------------------------------------------------------------------------
Enter the BlackBerry Developer Challenge  
This is your chance to win up to $100,000 in prizes! For a limited time, 
vendors submitting new applications to BlackBerry App World(TM) will have 
the opportunity to enter the BlackBerry Developer Challenge. See full prize 
details at: http://p.sf.net/sfu/blackberry
_______________________________________________
Musicpd-dev-team mailing list
Musicpd-dev-team@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/musicpd-dev-team

Reply via email to