Mailing-List:	contact cygwin-help AT cygwin DOT com; run by ezmlm
List-Subscribe:	<mailto:cygwin-subscribe AT cygwin DOT com>
List-Archive:	<http://sourceware.org/ml/cygwin/>
List-Post:	<mailto:cygwin AT cygwin DOT com>
List-Help:	<mailto:cygwin-help AT cygwin DOT com>, <http://sourceware.org/ml/#faqs>
Sender:	cygwin-owner AT cygwin DOT com
Mail-Followup-To:	cygwin AT cygwin DOT com
Delivered-To:	mailing list cygwin AT cygwin DOT com
Date:	Sun, 27 Feb 2005 11:54:22 -0500 (EST)
From:	Igor Pechtchanski <pechtcha AT cs DOT nyu DOT edu>
Reply-To:	cygwin AT cygwin DOT com
To:	cygwin AT cygwin DOT com
Subject:	Re: getopt required_argument (Was Re: regtool - 1.8 - Core dump)
In-Reply-To:	<Pine.GSO.4.61.0502271142170.14885@slinky.cs.nyu.edu>
Message-ID:	<Pine.GSO.4.61.0502271151530.14885@slinky.cs.nyu.edu>
References:	<20050227054557 DOT 45931 DOT qmail AT web51102 DOT mail DOT yahoo DOT com> <4221625B DOT 8D133A7D AT dessent DOT net> <Pine DOT GSO DOT 4 DOT 61 DOT 0502271142170 DOT 14885 AT slinky DOT cs DOT nyu DOT edu>
MIME-Version:	1.0

On Sun, 27 Feb 2005, Igor Pechtchanski wrote:

> On Sat, 26 Feb 2005, Brian Dessent wrote:
>
> > Alexander Joerg Herrmann wrote:
> >
> > > What I was mentioning is that
> > > regtool -K
> > > causes a core fault and something similar under windows because the
> > > argument is broken. I know that the argument is missing and so should
> > > regtool. It should produce a usage message insteed of a crash imo.
> >
> > The program should not core dump on invalid arguments.  That I agree
> > with, and it's a bug that should be fixed.
>
> Well, it's just a matter of adding a check for optarg being NULL and
> printing an appropriate message (e.g.,
>           if (optarg == NULL) {
>             fprintf(stderr, "Missing key-value separator\n");
>             usage();
>           }
> ).
>
> However, the question is what should getopt's behavior be if something
> specified as a "required_argument" is missing?  Should it simply set
> optarg to NULL, just like it does with "optional_argument", or should it
> return an error code of some sort?  Any getopt experts out there?

Whoops, never mind.  It's a bug, pure and simple.  According to
<http://refspecs.freestandards.org/LSB_2.0.0/LSB-generic/LSB-generic/libutil-getopt-3.html>,
two colons after a character indicate an *optional* argument, and for a
required argument one needs to use only one colon.  The only change needed
is that opts[] needs to only have one colon after "K".
	Igor
-- 
				http://cs.nyu.edu/~pechtcha/
      |\      _,,,---,,_		pechtcha AT cs DOT nyu DOT edu
ZZZzz /,`.-'`'    -.  ;-;;,_		igor AT watson DOT ibm DOT com
     |,4-  ) )-,_. ,\ (  `'-'		Igor Pechtchanski, Ph.D.
    '---''(_/--'  `-'\_) fL	a.k.a JaguaR-R-R-r-r-r-.-.-.  Meow!

"The Sun will pass between the Earth and the Moon tonight for a total
Lunar eclipse..." -- WCBS Radio Newsbrief, Oct 27 2004, 12:01 pm EDT

--
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple
Problem reports:       http://cygwin.com/problems.html
Documentation:         http://cygwin.com/docs.html
FAQ:                   http://cygwin.com/faq/

- Raw text -