Remove units from numbers

This is a first step to decouple formatting from information because of
two reasons:

 1. The components should only gather and return the values by design
 2. Fine grained user control should be a focus

Scaling will be implemented in a different way in a later commit.

diff --git a/components/battery.c b/components/battery.c
index 5d7a385..ff06f40 100644 (file)
--- a/components/battery.c
+++ b/components/battery.c
@@ -43,7 +43,7 @@
                        return NULL;
                }
 
                        return NULL;
                }
 

-               return bprintf("%d%%", perc);
+               return bprintf("%d", perc);

        }
 
        const char *
        }
 
        const char *


@@ -144,7 +144,7 @@
                struct apm_power_info apm_info;
 
                if (load_apm_power_info(&apm_info)) {
                struct apm_power_info apm_info;
 
                if (load_apm_power_info(&apm_info)) {

-                       return bprintf("%d%%", apm_info.battery_life);
+                       return bprintf("%d", apm_info.battery_life);

                }
 
                return NULL;
                }
 
                return NULL;

diff --git a/components/cpu.c b/components/cpu.c
index 53af71d..3fadb36 100644 (file)
--- a/components/cpu.c
+++ b/components/cpu.c
@@ -20,7 +20,7 @@
                        return NULL;
                }
 
                        return NULL;
                }
 

-               return fmt_human_10(freq * 1000, "Hz");
+               return fmt_human_10(freq * 1000);

        }
 
        const char *
        }
 
        const char *


@@ -39,7 +39,7 @@
                        return NULL;
                }
 
                        return NULL;
                }
 

-               return bprintf("%d%%", (int)(100 *
+               return bprintf("%d", (int)(100 *

                               ((b[0] + b[1] + b[2] + b[5] + b[6]) -
                                (a[0] + a[1] + a[2] + a[5] + a[6])) /
                               ((b[0] + b[1] + b[2] + b[3] + b[4] + b[5] + b[6]) -
                               ((b[0] + b[1] + b[2] + b[5] + b[6]) -
                                (a[0] + a[1] + a[2] + a[5] + a[6])) /
                               ((b[0] + b[1] + b[2] + b[3] + b[4] + b[5] + b[6]) -


@@ -67,7 +67,7 @@
                        return NULL;
                }
 
                        return NULL;
                }
 

-               return fmt_human_10((size_t)freq * 1000 * 1000, "Hz");
+               return fmt_human_10((size_t)freq * 1000 * 1000);

        }
 
        const char *
        }
 
        const char *


@@ -92,7 +92,7 @@
                        return NULL;
                }
 
                        return NULL;
                }
 

-               return bprintf("%d%%", 100 *
+               return bprintf("%d", 100 *

                               ((a[CP_USER] + a[CP_NICE] + a[CP_SYS] + a[CP_INTR]) -
                                (b[CP_USER] + b[CP_NICE] + b[CP_SYS] + b[CP_INTR])) /
                               ((a[CP_USER] + a[CP_NICE] + a[CP_SYS] + a[CP_INTR] +
                               ((a[CP_USER] + a[CP_NICE] + a[CP_SYS] + a[CP_INTR]) -
                                (b[CP_USER] + b[CP_NICE] + b[CP_SYS] + b[CP_INTR])) /
                               ((a[CP_USER] + a[CP_NICE] + a[CP_SYS] + a[CP_INTR] +

diff --git a/components/disk.c b/components/disk.c
index 998ad47..cf3c099 100644 (file)
--- a/components/disk.c
+++ b/components/disk.c
@@ -16,7 +16,7 @@ disk_free(const char *mnt)
                return NULL;
        }
 
                return NULL;
        }
 

-       return fmt_human_2(fs.f_frsize * fs.f_bavail, "B");
+       return fmt_human_2(fs.f_frsize * fs.f_bavail);

 }
 
 const char *
 }
 
 const char *


@@ -29,7 +29,7 @@ disk_perc(const char *mnt)
                return NULL;
        }
 
                return NULL;
        }
 

-       return bprintf("%d%%", (int)(100 *
+       return bprintf("%d", (int)(100 *

                       (1.0f - ((float)fs.f_bavail / (float)fs.f_blocks))));
 }
 
                       (1.0f - ((float)fs.f_bavail / (float)fs.f_blocks))));
 }
 


@@ -43,7 +43,7 @@ disk_total(const char *mnt)
                return NULL;
        }
 
                return NULL;
        }
 

-       return fmt_human_2(fs.f_frsize * fs.f_blocks, "B");
+       return fmt_human_2(fs.f_frsize * fs.f_blocks);

 }
 
 const char *
 }
 
 const char *


@@ -56,5 +56,5 @@ disk_used(const char *mnt)
                return NULL;
        }
 
                return NULL;
        }
 

-       return fmt_human_2(fs.f_frsize * (fs.f_blocks - fs.f_bfree), "B");
+       return fmt_human_2(fs.f_frsize * (fs.f_blocks - fs.f_bfree));

 }
 }

diff --git a/components/netspeeds.c b/components/netspeeds.c
index 14b7799..76557e0 100644 (file)
--- a/components/netspeeds.c
+++ b/components/netspeeds.c
@@ -29,8 +29,7 @@
                        return NULL;
                }
 
                        return NULL;
                }
 

-               return fmt_human_2((rxbytes - oldrxbytes) *
-                                  1000 / interval, "B/s");
+               return fmt_human_2((rxbytes - oldrxbytes) * 1000 / interval);

        }
 
        const char *
        }
 
        const char *


@@ -55,8 +54,7 @@
                        return NULL;
                }
 
                        return NULL;
                }
 

-               return fmt_human_2((txbytes - oldtxbytes) *
-                                  1000 / interval, "B/s");
+               return fmt_human_2((txbytes - oldtxbytes) * 1000 / interval);

        }
 #elif defined(__OpenBSD__)
        #include <string.h>
        }
 #elif defined(__OpenBSD__)
        #include <string.h>


@@ -97,8 +95,7 @@
                        return NULL;
                }
 
                        return NULL;
                }
 

-               return fmt_human_2((rxbytes - oldrxbytes) *
-                                  1000 / interval, "B/s");
+               return fmt_human_2((rxbytes - oldrxbytes) * 1000 / interval);

        }
 
        const char *
        }
 
        const char *


@@ -133,7 +130,6 @@
                        return NULL;
                }
 
                        return NULL;
                }
 

-               return fmt_human_2((txbytes - oldtxbytes) *
-                                  1000 / interval, "B/s");
+               return fmt_human_2((txbytes - oldtxbytes) * 1000 / interval);

        }
 #endif
        }
 #endif

diff --git a/components/ram.c b/components/ram.c
index 7c0870c..f653156 100644 (file)
--- a/components/ram.c
+++ b/components/ram.c
@@ -17,7 +17,7 @@
                        return NULL;
                }
 
                        return NULL;
                }
 

-               return fmt_human_2(free * 1024, "B");
+               return fmt_human_2(free * 1024);

        }
 
        const char *
        }
 
        const char *


@@ -34,8 +34,8 @@
                        return NULL;
                }
 
                        return NULL;
                }
 

-               return bprintf("%d%%", 100 * ((total - free) -
-                                             (buffers + cached)) / total);
+               return bprintf("%d", 100 * ((total - free) -
+                                           (buffers + cached)) / total);

        }
 
        const char *
        }
 
        const char *


@@ -48,7 +48,7 @@
                        return NULL;
                }
 
                        return NULL;
                }
 

-               return fmt_human_2(total * 1024, "B");
+               return fmt_human_2(total * 1024);

        }
 
        const char *
        }
 
        const char *


@@ -65,8 +65,7 @@
                        return NULL;
                }
 
                        return NULL;
                }
 

-               return fmt_human_2((total - free - buffers - cached) * 1024,
-                                  "B");
+               return fmt_human_2((total - free - buffers - cached) * 1024);

        }
 #elif defined(__OpenBSD__)
        #include <stdlib.h>
        }
 #elif defined(__OpenBSD__)
        #include <stdlib.h>


@@ -96,7 +95,8 @@
 
                if (load_uvmexp(&uvmexp)) {
                        free_pages = uvmexp.npages - uvmexp.active;
 
                if (load_uvmexp(&uvmexp)) {
                        free_pages = uvmexp.npages - uvmexp.active;

-                       return fmt_human_2(pagetok(free_pages, uvmexp.pageshift) * 1024, "B");
+                       return fmt_human_2(pagetok(free_pages,
+                                                  uvmexp.pageshift) * 1024);

                }
 
                return NULL;
                }
 
                return NULL;


@@ -110,7 +110,7 @@
 
                if (load_uvmexp(&uvmexp)) {
                        percent = uvmexp.active * 100 / uvmexp.npages;
 
                if (load_uvmexp(&uvmexp)) {
                        percent = uvmexp.active * 100 / uvmexp.npages;

-                       return bprintf("%d%%", percent);
+                       return bprintf("%d", percent);

                }
 
                return NULL;
                }
 
                return NULL;


@@ -122,7 +122,8 @@
                struct uvmexp uvmexp;
 
                if (load_uvmexp(&uvmexp)) {
                struct uvmexp uvmexp;
 
                if (load_uvmexp(&uvmexp)) {

-                       return fmt_human_2(pagetok(uvmexp.npages, uvmexp.pageshift) * 1024, "B");
+                       return fmt_human_2(pagetok(uvmexp.npages,
+                                                  uvmexp.pageshift) * 1024);

                }
 
                return NULL;
                }
 
                return NULL;


@@ -134,7 +135,8 @@
                struct uvmexp uvmexp;
 
                if (load_uvmexp(&uvmexp)) {
                struct uvmexp uvmexp;
 
                if (load_uvmexp(&uvmexp)) {

-                       return fmt_human_2(pagetok(uvmexp.active, uvmexp.pageshift) * 1024, "B");
+                       return fmt_human_2(pagetok(uvmexp.active,
+                                                  uvmexp.pageshift) * 1024);

                }
 
                return NULL;
                }
 
                return NULL;

diff --git a/components/swap.c b/components/swap.c
index f5db667..c697d61 100644 (file)
--- a/components/swap.c
+++ b/components/swap.c
@@ -48,7 +48,7 @@
                }
                sscanf(match, "SwapFree: %ld kB\n", &free);
 
                }
                sscanf(match, "SwapFree: %ld kB\n", &free);
 

-               return fmt_human_2(free * 1024, "B");
+               return fmt_human_2(free * 1024);

        }
 
        const char *
        }
 
        const char *


@@ -80,7 +80,7 @@
                        return NULL;
                }
 
                        return NULL;
                }
 

-               return bprintf("%d%%", 100 * (total - free - cached) / total);
+               return bprintf("%d", 100 * (total - free - cached) / total);

        }
 
        const char *
        }
 
        const char *


@@ -98,7 +98,7 @@
                }
                sscanf(match, "SwapTotal: %ld kB\n", &total);
 
                }
                sscanf(match, "SwapTotal: %ld kB\n", &total);
 

-               return fmt_human_2(total * 1024, "B");
+               return fmt_human_2(total * 1024);

        }
 
        const char *
        }
 
        const char *


@@ -126,7 +126,7 @@
                }
                sscanf(match, "SwapFree: %ld kB\n", &free);
 
                }
                sscanf(match, "SwapFree: %ld kB\n", &free);
 

-               return fmt_human_2((total - free - cached) * 1024, "B");
+               return fmt_human_2((total - free - cached) * 1024);

        }
 #elif defined(__OpenBSD__)
        #include <stdlib.h>
        }
 #elif defined(__OpenBSD__)
        #include <stdlib.h>


@@ -178,7 +178,7 @@
 
                getstats(&total, &used);
 
 
                getstats(&total, &used);
 

-               return fmt_human_2((total - used) * 1024, "B");
+               return fmt_human_2((total - used) * 1024);

        }
 
        const char *
        }
 
        const char *


@@ -192,7 +192,7 @@
                        return NULL;
                }
 
                        return NULL;
                }
 

-               return bprintf("%d%%", 100 * used / total);
+               return bprintf("%d", 100 * used / total);

        }
 
        const char *
        }
 
        const char *


@@ -202,7 +202,7 @@
 
                getstats(&total, &used);
 
 
                getstats(&total, &used);
 

-               return fmt_human_2(total * 1024, "B");
+               return fmt_human_2(total * 1024);

        }
 
        const char *
        }
 
        const char *


@@ -212,6 +212,6 @@
 
                getstats(&total, &used);
 
 
                getstats(&total, &used);
 

-               return fmt_human_2(used * 1024, "B");
+               return fmt_human_2(used * 1024);

        }
 #endif
        }
 #endif

diff --git a/components/temperature.c b/components/temperature.c
index 7e5cbe7..4e27a9d 100644 (file)
--- a/components/temperature.c
+++ b/components/temperature.c
@@ -13,7 +13,7 @@
                        return NULL;
                }
 
                        return NULL;
                }
 

-               return bprintf("%d°C", temp / 1000);
+               return bprintf("%d", temp / 1000);

        }
 #elif defined(__OpenBSD__)
        #include <errno.h>
        }
 #elif defined(__OpenBSD__)
        #include <errno.h>


@@ -44,6 +44,6 @@
                }
 
                /* kelvin to celsius */
                }
 
                /* kelvin to celsius */

-               return bprintf("%d°C", (temp.value - 273150000) / 1000000);
+               return bprintf("%d", (temp.value - 273150000) / 1000000);

        }
 #endif
        }
 #endif

diff --git a/components/volume.c b/components/volume.c
index 0bfbe08..8674211 100644 (file)
--- a/components/volume.c
+++ b/components/volume.c
@@ -42,5 +42,5 @@ vol_perc(const char *card)
 
        close(afd);
 
 
        close(afd);
 

-       return bprintf("%d%%", v & 0xff);
+       return bprintf("%d", v & 0xff);

 }
 }

diff --git a/components/wifi.c b/components/wifi.c
index 389adab..b3e1723 100644 (file)
--- a/components/wifi.c
+++ b/components/wifi.c
@@ -60,7 +60,7 @@
                sscanf(datastart + 1, " %*d   %d  %*d  %*d\t\t  %*d\t   "
                       "%*d\t\t%*d\t\t %*d\t  %*d\t\t %*d", &cur);
 
                sscanf(datastart + 1, " %*d   %d  %*d  %*d\t\t  %*d\t   "
                       "%*d\t\t%*d\t\t %*d\t  %*d\t\t %*d", &cur);
 

-               return bprintf("%d%%", (int)((float)cur / total * 100));
+               return bprintf("%d", (int)((float)cur / total * 100));

        }
 
        const char *
        }
 
        const char *


@@ -147,7 +147,7 @@
                                q = nr.nr_rssi >= -50 ? 100 : (nr.nr_rssi <= -100 ? 0 :
                                (2 * (nr.nr_rssi + 100)));
                        }
                                q = nr.nr_rssi >= -50 ? 100 : (nr.nr_rssi <= -100 ? 0 :
                                (2 * (nr.nr_rssi + 100)));
                        }

-                       return bprintf("%d%%", q);
+                       return bprintf("%d", q);

                }
 
                return NULL;
                }
 
                return NULL;

diff --git a/util.c b/util.c
index 5936a6d..6dd08d7 100644 (file)
--- a/util.c
+++ b/util.c
@@ -87,7 +87,7 @@ bprintf(const char *fmt, ...)
 }
 
 const char *
 }
 
 const char *

-fmt_human_2(size_t num, char *unit)
+fmt_human_2(size_t num)

 {
        size_t i;
        double scaled;
 {
        size_t i;
        double scaled;


@@ -99,11 +99,11 @@ fmt_human_2(size_t num, char *unit)
                scaled /= 1024.0;
        }
 
                scaled /= 1024.0;
        }
 

-       return bprintf("%.1f%s%s", scaled, prefix[i], unit);
+       return bprintf("%.1f%s", scaled, prefix[i]);

 }
 
 const char *
 }
 
 const char *

-fmt_human_10(size_t num, char *unit)
+fmt_human_10(size_t num)

 {
        size_t i;
        double scaled;
 {
        size_t i;
        double scaled;


@@ -115,7 +115,7 @@ fmt_human_10(size_t num, char *unit)
                scaled /= 1000.0;
        }
 
                scaled /= 1000.0;
        }
 

-       return bprintf("%.1f%s%s", scaled, prefix[i], unit);
+       return bprintf("%.1f%s", scaled, prefix[i]);

 }
 
 int
 }
 
 int

diff --git a/util.h b/util.h
index 64e30d0..4879414 100644 (file)
--- a/util.h
+++ b/util.h
@@ -10,6 +10,6 @@ void die(const char *, ...);
 
 int esnprintf(char *str, size_t size, const char *fmt, ...);
 const char *bprintf(const char *fmt, ...);
 
 int esnprintf(char *str, size_t size, const char *fmt, ...);
 const char *bprintf(const char *fmt, ...);

-const char *fmt_human_2(size_t num, char *unit);
-const char *fmt_human_10(size_t num, char *unit);
+const char *fmt_human_2(size_t num);
+const char *fmt_human_10(size_t num);

 int pscanf(const char *path, const char *fmt, ...);
 int pscanf(const char *path, const char *fmt, ...);