suckless/slstatus

battery: Remove unnecessary defines for paths (609ef3aa1f6c0a47822d425dc6bb0744042ca038)
Repositories | README | LICENSE
commit 609ef3aa1f6c0a47822d425dc6bb0744042ca038
parent 16e15217c6cb56bd4125c54e645b932a50655b9d
Author: Aaron Marcher <[email protected]>
Date:   Fri,  6 Jul 2018 23:55:18 +0200

battery: Remove unnecessary defines for paths

Diffstat:

Mcomponents/battery.c12+++++-------


1 file changed, 5 insertions(+), 7 deletions(-)

diff --git a/components/battery.c b/components/battery.c
@@ -8,11 +8,6 @@

 	#include <limits.h>
 	#include <unistd.h>
 
-	#define CHARGE_NOW "/sys/class/power_supply/%s/charge_now"
-	#define ENERGY_NOW "/sys/class/power_supply/%s/energy_now"
-	#define CURRENT_NOW "/sys/class/power_supply/%s/current_now"
-	#define POWER_NOW "/sys/class/power_supply/%s/power_now"
-
 	static const char *
 	pick(const char *bat, const char *f1, const char *f2, char *path,
 	     size_t length)
@@ -94,13 +89,16 @@

 			return NULL;
 		}
 
-		if (!pick(bat, CHARGE_NOW, ENERGY_NOW, path, sizeof(path)) ||
+		if (!pick(bat, "/sys/class/power_supply/%s/charge_now",
+		          "/sys/class/power_supply/%s/energy_now",
+		          path, sizeof(path)) ||
 		    pscanf(path, "%d", &charge_now) < 0) {
 			return NULL;
 		}
 
 		if (!strcmp(state, "Discharging")) {
-			if (!pick(bat, CURRENT_NOW, POWER_NOW, path,
+			if (!pick(bat, "/sys/class/power_supply/%s/current_now",
+			          "/sys/class/power_supply/%s/power_now", path,
 			          sizeof(path)) ||
 			    pscanf(path, "%d", &current_now) < 0) {
 				return NULL;