[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