Re: [gobo-eiffel-develop] Code Contribution

[Paul M. <pm...@re...>]

On Mon, Sep 15, 2008 at 02:33:47PM +0200, Bernd Schoeller wrote:
> Dear Paul,
>
> I have looked at your class text. The addition of an interval of valid  
> values for 'integer options' seems reasonable and should be included.  
> Still, I think it should be directly added to AP_INTEGER_OPTION, instead  
> of having a specific sub-class. It is just too small to justify a 
> complete new class. The default range is min-integer to max-integer, and 
> the help text will only display default ranges if the limits have been 
> changed.

Perfect.

> BTW: AP_INTEGER_INTERVAL_OPTION would be an option that takes an interval 
> as the parameter. (--foo=1..10 or -b 20-40). But that is probably too  
> specific for the library and not what you intended.

Agreed. I originally threw it together as BOUNDED_INTEGER_OPTION and 
then changed the name at the last minute since I used an INTEGER_INTERVAL 
in the definition.

-paul