From c040defd5f5cf5beed264dd057097625d9c0d423 Mon Sep 17 00:00:00 2001 From: M Stoeckl Date: Mon, 21 Jan 2019 12:46:45 -0500 Subject: [PATCH] Fix edge case bug in numlen, dropping use of math.h functions (Specifically, numlen when called with INT_MIN gave an incorrect result, because abs(INT_MIN) == INT_MIN < 0.) --- common/util.c | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/common/util.c b/common/util.c index 60f2f1606..bd7bed2dc 100644 --- a/common/util.c +++ b/common/util.c @@ -12,11 +12,12 @@ int wrap(int i, int max) { } int numlen(int n) { - if (n == 0) { - return 1; + int j = n <= 0 ? 1 : 0; + while (n) { + j++; + n /= 10; } - // Account for the '-' in negative numbers. - return log10(abs(n)) + (n > 0 ? 1 : 2); + return j; } uint32_t parse_color(const char *color) {