Mailing-List: contact dev-help@commons.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Commons Developers List" <dev@commons.apache.org>
Received-SPF: pass (athena.apache.org: domain of sebbaz@gmail.com designates
 209.85.212.177 as permitted sender)
MIME-Version: 1.0
In-Reply-To: 
 <CAJSjvwscHAcpJA1czC+16CD9-V=61=QwcJuErEN6y9muXZR_nQ@mail.gmail.com>
References: <20130130170551.BF19323888E7@eris.apache.org>
	<CAJSjvwscHAcpJA1czC+16CD9-V=61=QwcJuErEN6y9muXZR_nQ@mail.gmail.com>
Date: Wed, 30 Jan 2013 18:13:48 +0000
Message-ID: 
 <CAOGo0VZOu=CnGhYauqtp0Mqmv4pQYtqbSjEoC8_K5MFfhOA5CQ@mail.gmail.com>
Subject: Re: svn commit: r1440524 -
 /commons/proper/cli/trunk/src/main/java/org/apache/commons/cli/Options.java
From: sebb <sebbaz@gmail.com>
To: Commons Developers List <dev@commons.apache.org>
Content-Type: text/plain; charset=ISO-8859-1

On 30 January 2013 17:12, Thomas Neidhart <thomas.neidhart@gmail.com> wrote:
> On Wed, Jan 30, 2013 at 6:05 PM, <sebb@apache.org> wrote:
>
>> Author: sebb
>> Date: Wed Jan 30 17:05:51 2013
>> New Revision: 1440524
>>
>> URL: http://svn.apache.org/viewvc?rev=1440524&view=rev
>> Log:
>> Document unexpected list contents
>>
>> Modified:
>>
>> commons/proper/cli/trunk/src/main/java/org/apache/commons/cli/Options.java
>>
>> Modified:
>> commons/proper/cli/trunk/src/main/java/org/apache/commons/cli/Options.java
>> URL:
>> http://svn.apache.org/viewvc/commons/proper/cli/trunk/src/main/java/org/apache/commons/cli/Options.java?rev=1440524&r1=1440523&r2=1440524&view=diff
>>
>> ==============================================================================
>> ---
>> commons/proper/cli/trunk/src/main/java/org/apache/commons/cli/Options.java
>> (original)
>> +++
>> commons/proper/cli/trunk/src/main/java/org/apache/commons/cli/Options.java
>> Wed Jan 30 17:05:51 2013
>> @@ -55,6 +55,8 @@ public class Options implements Serializ
>>      private Map<String, Option> longOpts = new LinkedHashMap<String,
>> Option>();
>>
>>      /** a map of the required options */
>> +    // N.B. This can contain either a String (addOption) or an
>> OptionGroup (addOptionGroup)
>> +    // TODO this seems wrong
>>      private List<Object> requiredOpts = new ArrayList<Object>();
>>
>
> Indeed, I also spotted this and failed to resolve it, as the logic in the
> parsers is somehow taken advantage of it in a way I do not yet fully
> understand.

Me neither.

Maybe the code would still work if the entries were always OptionGroups.
This could perhaps be done by converting the Option into a
single-entry OptionGroup and storing that, rather than storing the
Option key String.
In theory that might work ...

> Thomas

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