[prev in list] [next in list] [prev in thread] [next in thread] 

List:       jakarta-commons-dev
Subject:    Re: [CLI] [patch] Parser bug
From:       John Keyes <jbjk () mac ! com>
Date:       2002-09-30 22:25:16
[Download RAW message or body]

Cheers Pete,  I'll add the unit test for it.

-John K

On Monday, Sep 30, 2002, at 16:30 Etc/GMT, MADDOCKS,PETE 
(HP-Loveland,ex1) wrote:

> Hi John,
>
> We ran across a bug in the case where a missing required argument for 
> an
> option was only caught if the option was at the end of command line.
>
> So for the command:
> -----------------
> SYNOPSIS
> ndmp [ options ]
> -o, --old-password oldPassword
> Use this option to specify the old password.
>
> -n, --new-password newPassword
> Use this option to specify the new password
> -----------------
> 1. If the user types:
>    ndmp -o oldpassword -n
>    Then the Parser class correctly throws a MissingArgumentException.
>
> 2. However, if the user types:
>    ndmp -o -n newpassword
>    Then the Parser class does not throw the Missing ArgumentException.
>
> So I have fixed this bug in the processArgs() method in the Parser 
> class.
>
> Please let me know if you have questions about it.
>
> Thanks,
> Peter Maddocks
> HP NSS NAS Software Design Engineer
> peter_maddocks@hp.com
>
>
>
> <Parser.java.patch>--
> To unsubscribe, e-mail:   
> <mailto:commons-dev-unsubscribe@jakarta.apache.org>
> For additional commands, e-mail: 
> <mailto:commons-dev-help@jakarta.apache.org>


--
To unsubscribe, e-mail:   <mailto:commons-dev-unsubscribe@jakarta.apache.org>
For additional commands, e-mail: <mailto:commons-dev-help@jakarta.apache.org>

[prev in list] [next in list] [prev in thread] [next in thread]