Remove units from numbers

[slstatus.git] / components / ram.c

diff --git a/components/ram.c b/components/ram.c
index 5611022..f653156 100644 (file)
--- a/components/ram.c
+++ b/components/ram.c
@@ -9,8 +9,15 @@
        {
                long free;
 
-               return (pscanf("/proc/meminfo", "MemFree: %ld kB\n", &free) == 1) ?
-                      bprintf("%f", (float)free / 1024 / 1024) : NULL;
+               if (pscanf("/proc/meminfo",
+                          "MemTotal: %ld kB\n"
+                          "MemFree: %ld kB\n"
+                          "MemAvailable: %ld kB\n",
+                          &free, &free, &free) != 3) {
+                       return NULL;
+               }
+
+               return fmt_human_2(free * 1024);
        }
 
        const char *
@@ -18,15 +25,17 @@
        {
                long total, free, buffers, cached;
 
-               return (pscanf("/proc/meminfo",
-                              "MemTotal: %ld kB\n"
-                              "MemFree: %ld kB\n"
-                              "MemAvailable: %ld kB\nBuffers: %ld kB\n"
-                              "Cached: %ld kB\n",
-                              &total, &free, &buffers, &buffers, &cached) == 5) ?
-                      bprintf("%d", 100 * ((total - free) - (buffers + cached)) /
-                              total) :
-                      NULL;
+               if (pscanf("/proc/meminfo",
+                          "MemTotal: %ld kB\n"
+                          "MemFree: %ld kB\n"
+                          "MemAvailable: %ld kB\nBuffers: %ld kB\n"
+                          "Cached: %ld kB\n",
+                          &total, &free, &buffers, &buffers, &cached) != 5) {
+                       return NULL;
+               }
+
+               return bprintf("%d", 100 * ((total - free) -
+                                           (buffers + cached)) / total);
        }
 
        const char *
@@ -34,8 +43,12 @@
        {
                long total;
 
-               return (pscanf("/proc/meminfo", "MemTotal: %ld kB\n", &total) == 1) ?
-                      bprintf("%f", (float)total / 1024 / 1024) : NULL;
+               if (pscanf("/proc/meminfo", "MemTotal: %ld kB\n",
+                          &total) != 1) {
+                       return NULL;
+               }
+
+               return fmt_human_2(total * 1024);
        }
 
        const char *
@@ -43,22 +56,26 @@
        {
                long total, free, buffers, cached;
 
-               return (pscanf("/proc/meminfo",
-                              "MemTotal: %ld kB\n"
-                              "MemFree: %ld kB\n"
-                              "MemAvailable: %ld kB\nBuffers: %ld kB\n"
-                              "Cached: %ld kB\n",
-                              &total, &free, &buffers, &buffers, &cached) == 5) ?
-                      bprintf("%f", (float)(total - free - buffers - cached) /
-                              1024 / 1024) :
-                      NULL;
+               if (pscanf("/proc/meminfo",
+                          "MemTotal: %ld kB\n"
+                          "MemFree: %ld kB\n"
+                          "MemAvailable: %ld kB\nBuffers: %ld kB\n"
+                          "Cached: %ld kB\n",
+                          &total, &free, &buffers, &buffers, &cached) != 5) {
+                       return NULL;
+               }
+
+               return fmt_human_2((total - free - buffers - cached) * 1024);
        }
 #elif defined(__OpenBSD__)
-       #include <sys/types.h>
-       #include <sys/sysctl.h>
        #include <stdlib.h>
+       #include <sys/sysctl.h>
+       #include <sys/types.h>
        #include <unistd.h>
 
+       #define LOG1024         10
+       #define pagetok(size, pageshift) ((size) << (pageshift - LOG1024))
+
        inline int
        load_uvmexp(struct uvmexp *uvmexp)
        {
@@ -74,14 +91,12 @@
        ram_free(void)
        {
                struct uvmexp uvmexp;
-               float free;
                int free_pages;
 
                if (load_uvmexp(&uvmexp)) {
                        free_pages = uvmexp.npages - uvmexp.active;
-                       free = (double) (free_pages * uvmexp.pagesize) / 1024 /
-                              1024 / 1024;
-                       return bprintf("%f", free);
+                       return fmt_human_2(pagetok(free_pages,
+                                                  uvmexp.pageshift) * 1024);
                }
 
                return NULL;
@@ -105,12 +120,10 @@
        ram_total(void)
        {
                struct uvmexp uvmexp;
-               float total;
 
                if (load_uvmexp(&uvmexp)) {
-                       total = (double) (uvmexp.npages * uvmexp.pagesize) /
-                               1024 / 1024 / 1024;
-                       return bprintf("%f", total);
+                       return fmt_human_2(pagetok(uvmexp.npages,
+                                                  uvmexp.pageshift) * 1024);
                }
 
                return NULL;
@@ -120,12 +133,10 @@
        ram_used(void)
        {
                struct uvmexp uvmexp;
-               float used;
 
                if (load_uvmexp(&uvmexp)) {
-                       used = (double) (uvmexp.active * uvmexp.pagesize) /
-                              1024 / 1024 / 1024;
-                       return bprintf("%f", used);
+                       return fmt_human_2(pagetok(uvmexp.active,
+                                                  uvmexp.pageshift) * 1024);
                }
 
                return NULL;