svn commit: r211304 - head/lib/libutil
brde at optusnet.com.au
Tue Aug 24 00:13:14 UTC 2010
On Mon, 23 Aug 2010, [utf-8] Dag-Erling SmÃ¸rgrav wrote:
> Dimitry Andric <dimitry at andric.com> writes:
>> Dag-Erling SmÃ¸rgrav <des at des.no> writes:
>>> Bruce Cran <bruce at cran.org.uk> writes:
>>>> Somewhat related, there are overflow bugs in humanize_number - for
>>>> example df(1) fails to display space from a 100PB filesystem
>>> Patch? :)
>> Attached. This makes humanize_number() work properly for all possible
>> int64_t values.
> That's awesome! Any way I can convince you to fix expand_number() as
> well? I got a detailed explanation of what's wrong with it (both before
> and after my commits) from bde@ (cc:ed) in private correspondence; I can
> forward it to you if he doesn't mind.
I think the final point was that it should go back to supporting signed
numbers (only), and that means int64_t ones until scientificize^dehumanize^W
humanize_number() is fixed to support intmax_t ones.
More information about the svn-src-all