From: Bernd S. <ber...@co...> - 2008-09-15 12:33:41
|
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. 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. Adding an AP_DATE_OPTION class seems very interesting, though it would add a dependency of the argument library to the 'time' library (which - AFAIK - is currently not there). Parsing dates from strings is also a complex job. Some of this work has been done in the XML libraries, but again we get into trouble of creating dependencies between libraries (there was a discussion between Colin and Eric sometime ago about this issues, but I do not know how this was resolved). Bernd On Thu, 11 Sep 2008 17:03:49 +0200, Eric Bezault <er...@us...> wrote: > Thanks Paul. > > Bernd, can you have a look at Paul's contribution and > see how to integrate it into your Arguments library? > Thanks. > -- Bernd Schoeller, PhD, CTO Comerge AG, Technoparkstrasse 1, CH-8005 Zurich, Switzerland |