[m-rev.] diff: .NET back-end: another string__to_float fix

Fergus Henderson fjh at cs.mu.OZ.AU
Tue Oct 22 17:13:31 AEST 2002


On 22-Oct-2002, Peter Moulder <pmoulder at csse.monash.edu.au> wrote:
> > +	// leading or trailing whitespace is not allowed
> > +	if (System::Char::IsWhiteSpace(FloatString, 0) ||
> > +	    System::Char::IsWhiteSpace(FloatString, FloatString->Length - 1))
> > +	{
> > +	    SUCCESS_INDICATOR = MR_FALSE;
> 
> Does the above work correctly (e.g. not throwing an exception) if
> FloatString is empty?

No.  Thanks for catching that.

I will fix it, and update the test case to test for it.

-- 
Fergus Henderson <fjh at cs.mu.oz.au>  |  "I have always known that the pursuit
The University of Melbourne         |  of excellence is a lethal habit"
WWW: <http://www.cs.mu.oz.au/~fjh>  |     -- the last words of T. S. Garp.
--------------------------------------------------------------------------
mercury-reviews mailing list
post:  mercury-reviews at cs.mu.oz.au
administrative address: owner-mercury-reviews at cs.mu.oz.au
unsubscribe: Address: mercury-reviews-request at cs.mu.oz.au Message: unsubscribe
subscribe:   Address: mercury-reviews-request at cs.mu.oz.au Message: subscribe
--------------------------------------------------------------------------



More information about the reviews mailing list