[git commit] uptime: more compatible output
Denys Vlasenko
vda.linux at googlemail.com
Wed Aug 10 11:00:04 UTC 2011
commit: http://git.busybox.net/busybox/commit/?id=5845a06c9ed8c16200942d1fba581db90a451cc6
branch: http://git.busybox.net/busybox/commit/?id=refs/heads/master
was: 12:59:35 up 1:57, 4 users, load average: 0.11, 0.20, 0.53
is: 12:59:37 up 1:57, 4 users, load average: 0.11, 0.20, 0.53
While at it, switched code to unsigned division.
Based on patch by Pere Orga <gotrunks at gmail.com>.
Signed-off-by: Denys Vlasenko <vda.linux at googlemail.com>
---
procps/uptime.c | 40 ++++++++++++++++++++--------------------
1 files changed, 20 insertions(+), 20 deletions(-)
diff --git a/procps/uptime.c b/procps/uptime.c
index c3a2740..778812a 100644
--- a/procps/uptime.c
+++ b/procps/uptime.c
@@ -48,15 +48,15 @@
#ifndef FSHIFT
# define FSHIFT 16 /* nr of bits of precision */
#endif
-#define FIXED_1 (1<<FSHIFT) /* 1.0 as fixed-point */
-#define LOAD_INT(x) ((x) >> FSHIFT)
-#define LOAD_FRAC(x) LOAD_INT(((x) & (FIXED_1-1)) * 100)
+#define FIXED_1 (1 << FSHIFT) /* 1.0 as fixed-point */
+#define LOAD_INT(x) (unsigned)((x) >> FSHIFT)
+#define LOAD_FRAC(x) LOAD_INT(((x) & (FIXED_1 - 1)) * 100)
int uptime_main(int argc, char **argv) MAIN_EXTERNALLY_VISIBLE;
int uptime_main(int argc UNUSED_PARAM, char **argv UNUSED_PARAM)
{
- int updays, uphours, upminutes;
+ unsigned updays, uphours, upminutes;
struct sysinfo info;
struct tm *current_time;
time_t current_secs;
@@ -66,32 +66,32 @@ int uptime_main(int argc UNUSED_PARAM, char **argv UNUSED_PARAM)
sysinfo(&info);
- printf(" %02d:%02d:%02d up ",
+ printf(" %02u:%02u:%02u up ",
current_time->tm_hour, current_time->tm_min, current_time->tm_sec);
- updays = (int) info.uptime / (60*60*24);
+ updays = (unsigned) info.uptime / (unsigned)(60*60*24);
if (updays)
- printf("%d day%s, ", updays, (updays != 1) ? "s" : "");
- upminutes = (int) info.uptime / 60;
- uphours = (upminutes / 60) % 24;
+ printf("%u day%s, ", updays, (updays != 1) ? "s" : "");
+ upminutes = (unsigned) info.uptime / (unsigned)60;
+ uphours = (upminutes / (unsigned)60) % (unsigned)24;
upminutes %= 60;
if (uphours)
- printf("%2d:%02d, ", uphours, upminutes);
+ printf("%2u:%02u", uphours, upminutes);
else
- printf("%d min, ", upminutes);
+ printf("%u min", upminutes);
#if ENABLE_FEATURE_UPTIME_UTMP_SUPPORT
-{
- struct utmp *ut;
- int users = 0;
- while ((ut = getutent())) {
- if ((ut->ut_type == USER_PROCESS) && (ut->ut_name[0] != '\0'))
- users++;
+ {
+ struct utmp *ut;
+ unsigned users = 0;
+ while ((ut = getutent()) != NULL) {
+ if ((ut->ut_type == USER_PROCESS) && (ut->ut_name[0] != '\0'))
+ users++;
+ }
+ printf(", %u users", users);
}
- printf("%d users, ", users);
-}
#endif
- printf("load average: %ld.%02ld, %ld.%02ld, %ld.%02ld\n",
+ printf(", load average: %u.%02u, %u.%02u, %u.%02u\n",
LOAD_INT(info.loads[0]), LOAD_FRAC(info.loads[0]),
LOAD_INT(info.loads[1]), LOAD_FRAC(info.loads[1]),
LOAD_INT(info.loads[2]), LOAD_FRAC(info.loads[2]));
--
1.7.3.4
More information about the busybox-cvs
mailing list