On 09/15/10 16:50, Juan Quintela wrote:
> jes.soren...@redhat.com wrote:
>> From: Jes Sorensen <jes.soren...@redhat.com>
>> value <<= 10;
>> break;
>> case 0:
>> + if (divider) {
>> + value = 0;
>> + break;
>
> changing break by goto fail here?
> 1.5G and 1.0G is ok, but using 1024.00 or similar should be one error,
> no?
>
> nice cleanup for the rest of the patch series.In my testing, 1234.5 fails as expected, so I expect 1024.00 to fail as well. Cheers, Jes
