On 11/21/18 4:44 AM, David Hildenbrand wrote:
@@ -206,20 +206,18 @@ static int64_t suffix_mul(char suffix, int64_t unit)
* in *end, if not NULL. Return -ERANGE on overflow, Return -EINVAL on
Pre-existing, but since you're touching this area: the second 'Return'
is unusual capitalization
David Hildenbrand writes:
> On 20.11.18 21:41, Eric Blake wrote:
>> On 11/20/18 2:31 PM, Markus Armbruster wrote:
>>> Eric Blake writes:
>>>
On 11/20/18 3:25 AM, David Hildenbrand wrote:
> qemu_strtosz() & friends reject NaNs, but happily accept inifities.
On 20.11.18 21:41, Eric Blake wrote:
> On 11/20/18 2:31 PM, Markus Armbruster wrote:
>> Eric Blake writes:
>>
>>> On 11/20/18 3:25 AM, David Hildenbrand wrote:
qemu_strtosz() & friends reject NaNs, but happily accept inifities.
>>>
>>> s/inifities/infinities/
>>>
They shouldn't. Fix
On 11/20/18 2:31 PM, Markus Armbruster wrote:
Eric Blake writes:
On 11/20/18 3:25 AM, David Hildenbrand wrote:
qemu_strtosz() & friends reject NaNs, but happily accept inifities.
s/inifities/infinities/
They shouldn't. Fix that.
The fix makes use of qemu_strtod_finite(). To avoid ugly
Eric Blake writes:
> On 11/20/18 3:25 AM, David Hildenbrand wrote:
>> qemu_strtosz() & friends reject NaNs, but happily accept inifities.
>
> s/inifities/infinities/
>
>> They shouldn't. Fix that.
>>
>> The fix makes use of qemu_strtod_finite(). To avoid ugly casts,
>> change the @end parameter
On 11/20/18 3:25 AM, David Hildenbrand wrote:
qemu_strtosz() & friends reject NaNs, but happily accept inifities.
s/inifities/infinities/
They shouldn't. Fix that.
The fix makes use of qemu_strtod_finite(). To avoid ugly casts,
change the @end parameter of qemu_strtosz() & friends from char