Re: Code Contribution

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

-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/

_______________________________________________
gobo-eiffel-develop mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/gobo-eiffel-develop

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.7 (GNU/Linux)

iD8DBQFIzq1OfSBtZvWed8IRAlehAJsHR0CPAQMz/xsOirDpvrxStiH/cgCfVVcr
BEcD5uukiRPjoh8iPJXw7Vc=
=QUBX
-----END PGP SIGNATURE-----