battery: Consistent naming for capacity percentage
authorplanet36 <planet36@users.noreply.github.com>
Tue, 6 Apr 2021 16:48:18 +0000 (12:48 -0400)
committerdrkhsh <me@drkhsh.at>
Mon, 19 Dec 2022 01:44:21 +0000 (02:44 +0100)
https://www.kernel.org/doc/html/latest/power/power_supply_class.html

Co-authored-by: drkhsh <me@drkhsh.at>
Signed-off-by: drkhsh <me@drkhsh.at>
components/battery.c

index 9fa1014e01ba4f1050798e3d540c0aaf9e000941..1c753f903cde3de224ce37b4485de6693369790a 100644 (file)
@@ -6,6 +6,9 @@
 #include "../util.h"
 
 #if defined(__linux__)
+/*
+ * https://www.kernel.org/doc/html/latest/power/power_supply_class.html
+ */
        #include <limits.h>
        #include <stdint.h>
        #include <unistd.h>
        const char *
        battery_perc(const char *bat)
        {
-               int perc;
+               int cap_perc;
                char path[PATH_MAX];
 
                if (esnprintf(path, sizeof(path), POWER_SUPPLY_CAPACITY, bat) < 0)
                        return NULL;
-               if (pscanf(path, "%d", &perc) != 1)
+               if (pscanf(path, "%d", &cap_perc) != 1)
                        return NULL;
 
-               return bprintf("%d", perc);
+               return bprintf("%d", cap_perc);
        }
 
        const char *
        const char *
        battery_perc(const char *unused)
        {
-               int cap;
+               int cap_perc;
                size_t len;
 
-               len = sizeof(cap);
-               if (sysctlbyname(BATTERY_LIFE, &cap, &len, NULL, 0) < 0 || !len)
+               len = sizeof(cap_perc);
+               if (sysctlbyname(BATTERY_LIFE, &cap_perc, &len, NULL, 0) < 0 || !len)
                        return NULL;
 
-               return bprintf("%d", cap);
+               return bprintf("%d", cap_perc);
        }
 
        const char *