----- Original Message ----- From: "Jaroslaw Kowalski" <[EMAIL PROTECTED]> To: "Gert Driesen" <[EMAIL PROTECTED]>; "! nant" <[EMAIL PROTECTED]> Sent: Monday, February 23, 2004 8:22 AM Subject: Re: [nant-dev] PATCH: DateTime and TimeSpan literals
> > Hi Jarek, > > > > I agree that this looks more readable, but I'm not sure its good to add > > another way to create datetime or timespan instances. We'll need to > retain > > the convert::to-datetime/timespan anyway ... > > convert::to-datetime is here to stay (for example to convert a value > retrieved from a property), but they are more formal right now - they only > accept one of the mentioned formats, nothing else. I'd rather not have two ways of accomplishing the same thing ? If we can convert from string to datetime we have all we need, no ? > > > We should also stick to using the .NET framework conversion routines to > > handle conversion to and from datetime/timespan. Maybe one day we'll > allow > > users to define the culture in which they want to run NAnt ... > > The conversion routines in .NET aren't as strict as possible (vide > boolean=>double) and looks like MS does them differently than mono in some > cases. We could either file one mono bug after another or just implement our > own conversion functions. I'd say file bug reports. > > That's what my patch does. > > Do you really want to allow the cultures to be specified? Why? This kills > portability IMHO. No don't think we need to support this, bad example ... > > > Again, I agree that this is not very readably, but I think this is a > problem > > with the expression support as such ... > > ;-( What can we do to improve it? nothing for now, I guess ... that are other things we need to look at first ... Gert ------------------------------------------------------- SF.Net is sponsored by: Speed Start Your Linux Apps Now. Build and deploy apps & Web services for Linux with a free DVD software kit from IBM. Click Now! http://ads.osdn.com/?ad_id=1356&alloc_id=3438&op=click _______________________________________________ nant-developers mailing list [EMAIL PROTECTED] https://lists.sourceforge.net/lists/listinfo/nant-developers
