I have a version that uses getopt_long but I thought I was told that was a
bad idea because it comes from GNU getopt.h and we couldn't count on that
being part of embedded systems?  I can dig and find the email if it's
relevant.

Sorry about the debug code!  I pulled that out of it on my end and must have
forgotten to push it up to the repo.

Jeffrey

On Tue, Jul 7, 2009 at 11:06 AM, Max Kellermann <m...@duempel.org> wrote:

> 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