From b7cbbd6f018241962b432c1a608067614d223a6d Mon Sep 17 00:00:00 2001 From: Paul Eggert Date: Tue, 22 Mar 2011 01:15:43 -0700 Subject: [PATCH] * xdisp.c (pint2hrstr): Simplify and tune code slightly. This also avoids a (bogus) GCC warning with gcc -Wstrict-overflow. --- src/ChangeLog | 2 ++ src/xdisp.c | 5 ++--- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/src/ChangeLog b/src/ChangeLog index a9f3e3a5b70..fd135de123b 100644 --- a/src/ChangeLog +++ b/src/ChangeLog @@ -4,6 +4,8 @@ This is less likely to overflow, and avoids undefined behavior if overflow does occur. All callers changed. Use strtoul to scan for the unsigned long integer. + (pint2hrstr): Simplify and tune code slightly. + This also avoids a (bogus) GCC warning with gcc -Wstrict-overflow. * scroll.c (do_scrolling): Work around GCC bug 48228. See . diff --git a/src/xdisp.c b/src/xdisp.c index 93bc8c9479e..457a5e3e11b 100644 --- a/src/xdisp.c +++ b/src/xdisp.c @@ -19006,7 +19006,7 @@ pint2str (register char *buf, register int width, register EMACS_INT d) static const char power_letter[] = { - 0, /* not used */ + 0, /* no letter */ 'k', /* kilo */ 'M', /* mega */ 'G', /* giga */ @@ -19088,8 +19088,7 @@ pint2hrstr (char *buf, int width, int d) p = psuffix = buf + max (width, length); /* Print EXPONENT. */ - if (exponent) - *psuffix++ = power_letter[exponent]; + *psuffix++ = power_letter[exponent]; *psuffix = '\0'; /* Print TENTHS. */ -- 2.39.5