[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: numfmt: minor improvements to --field parsing
From: |
Pádraig Brady |
Subject: |
Re: numfmt: minor improvements to --field parsing |
Date: |
Sat, 12 Sep 2015 10:41:52 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.6.0 |
On 12/09/15 04:00, Assaf Gordon wrote:
> On Thu, Sep 10, 2015 at 03:53:38PM +0100, Pádraig Brady wrote:
>> Usually you would use SIZE_OFLOW in this context,
>> but here SIZE_MAX is used internally to represent an open ended range,
>> thus is not allowed as input. Therefore SIZE_MAX is more correct here.
>
> I see, thanks for catching that (I now see you did explictly write
> SIZE_MAX in the previous email, sorry for missing it).
>
> The attached patch uses SIZE_MAX instead of SIZE_OFLOW,
> and also adds some more comments in 'set-fields()' to about this.
+1, please push.
thanks!
Pádraig.