struts-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "David Graham" <dgraham1...@hotmail.com>
Subject RE: Would a fix be useful
Date Tue, 01 Apr 2003 22:14:39 GMT
Couldn't a simple range check on the number be performed first, cutting down 
on the number of times an exception would be thrown?

David



>From: "Karr, David" <David.Karr@wamu.net>
>Reply-To: "Struts Developers List" <struts-dev@jakarta.apache.org>
>To: Struts Developers List <struts-dev@jakarta.apache.org>
>Subject: RE: Would a fix be useful
>Date: Tue, 01 Apr 2003 14:11:04 -0800
>
>I would point out a couple things about this issue, however.  If you
>inspect the code in "Double.parseDouble()" and "Long.parseLong()", it
>becomes clear that a complete implementation of this is not trivial.
>Also note that the "commons-lang" project apparently addresses some of
>this, but I don't how well it avoids exceptions for normal processing.
>
> > -----Original Message-----
> > From: David Graham [mailto:dgraham1980@hotmail.com]
> >
> > Put your function in the compare tag code.  Submit a patch in cvs diff
>-u
> > format to the bug report you opened.
> >
> > David
> >
> > >From: Brendan.Johnston@WellsFargo.COM
> > >
> > >I created bug http://nagoya.apache.org/bugzilla/show_bug.cgi?id=18583
> > >
> > >If I coded a fix would that be useful?
> > >
> > >What should my expectations around getting a fix validated and
>commited
> > be?
> > >
> > >Given that a fix would involve some utility type functionality
> > >(isLong,isDouble)),
> > >is there a place where that should go (org.apache.struts.util)?
> > >
> > >Or should I just shove it in the Compare class file?
> > >
> > >Or is there a related project this should go in?
>
>---------------------------------------------------------------------
>To unsubscribe, e-mail: struts-dev-unsubscribe@jakarta.apache.org
>For additional commands, e-mail: struts-dev-help@jakarta.apache.org
>


_________________________________________________________________
The new MSN 8: advanced junk mail protection and 2 months FREE*  
http://join.msn.com/?page=features/junkmail


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


Mime
View raw message