[m-rev.] for review: overflow in string-to-float conversions

Julien Fischer jfischer at opturion.com
Mon Dec 22 14:33:04 AEDT 2014


On Mon, 22 Dec 2014, Paul Bone wrote:

> On Mon, Dec 22, 2014 at 12:36:40PM +1100, Julien Fischer wrote:
>>
>> For review by anyone.
>>
>> diff --git a/library/string.m b/library/string.m
>> index ed8b7ce..d41147e 100644
>> --- a/library/string.m
>> +++ b/library/string.m
>> @@ -1098,13 +1098,15 @@
>>      %
>>  :- func det_base_string_to_int(int, string) = int.
>>
>> -    % Convert a string to a float. If the string is not a syntactically
>> -    % correct float literal, to_float fails.
>> +    % Convert a string to a float, returning infinity or -infinity if the
>> +    % conversion overflows.  Fails if the string is not a syntactically correct
>> +    % float literal.
>>      %
>>  :- pred to_float(string::in, float::out) is semidet.
>>
>> -    % Convert a string to a float. Throws an exception if the string is not
>> -    % a syntactically correct float literal.
>> +    % Convert a string to a float, returning infinity or -infinity if the
>> +    % conversion overflows.  Fails if the string is not a syntactically correct
>> +    % float literal.
>>      %
>>  :- func det_to_float(string) = float.
>>
>
> You have a copy-pasted error.  This does not fail, it is det.

Fixed.

Cheers,
Julien.



More information about the reviews mailing list