From 7850f1e2ac9ef6e9c859c86e35e2265f6367ab20 Mon Sep 17 00:00:00 2001 From: Eric Andersen Date: Wed, 7 Mar 2001 03:50:57 +0000 Subject: Rewrite of format so it behaves itself -Erik --- utility.c | 52 ++++++++++++++++------------------------------------ 1 file changed, 16 insertions(+), 36 deletions(-) (limited to 'utility.c') diff --git a/utility.c b/utility.c index 2d27596e0..0da98f13d 100644 --- a/utility.c +++ b/utility.c @@ -1797,43 +1797,23 @@ ssize_t safe_read(int fd, void *buf, size_t count) #ifdef BB_FEATURE_HUMAN_READABLE const char *format(unsigned long val, unsigned long hr) { - static const char strings[] = { '0', 0, 'k', 0, 'M', 0, 'G', 0 }; - static const char fmt[] = "%lu"; - static const char fmt_u[] = "%lu.%lu%s"; - - static char str[10]; - - unsigned long frac __attribute__ ((unused)); /* 'may be uninitialized' warning is ok */ - const char *u; - const char *f; - -#if 1 - if(val == 0) { /* This may be omitted to reduce size */ - return strings; /* at the cost of speed. */ - } -#endif - - u = strings; - f = fmt; - if (hr) { - val /= hr; - } else { - while ((val >= KILOBYTE) && (*u != 'G')) { - f = fmt_u; - u += 2; - frac = (((val % KILOBYTE) * 10) + (KILOBYTE/2)) / KILOBYTE; - val /= KILOBYTE; - if (frac >= 10) { /* We need to round up here. */ - ++val; - frac = 0; - } - } + int i=0; + static char str[10] = "\0"; + static const char strings[] = { 'k', 'M', 'G', 'T', 0 }; + unsigned long divisor = 1; + + if(val == 0) + return("0"); + if(hr) + snprintf(str, 9, "%ld", val/hr); + else { + while(val >= divisor && i <= 4) { + divisor=divisor<<10, i++; + } + divisor=divisor>>10, i--; + snprintf(str, 9, "%.1Lf%c", (long double)(val)/divisor, strings[i]); } - - /* If f==fmt then 'frac' and 'u' are ignored and need not be set. */ - snprintf(str, sizeof(str), f, val, frac, u); - - return str; + return(str); } #endif -- cgit v1.2.3