From 5e33d9fe748efc8df49c123fe01d486b04c20f57 Mon Sep 17 00:00:00 2001 From: John Baldwin Date: Thu, 8 Oct 2015 13:15:38 -0700 Subject: [PATCH] Properly detect the battery charging status under FreeBSD. The hw.acpi.battery.state sysctl returns a bitmask of flags as defined in . Use constants from this header to examine the state and check for the charging flag to determine if the battery is charging. --- src/print_battery_info.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/print_battery_info.c b/src/print_battery_info.c index f65090a..90db4bb 100644 --- a/src/print_battery_info.c +++ b/src/print_battery_info.c @@ -12,6 +12,7 @@ #if defined(__FreeBSD__) || defined(__FreeBSD_kernel__) || defined(__DragonFly__) #include #include +#include #endif #if defined(__OpenBSD__) @@ -236,7 +237,7 @@ void print_battery_info(yajl_gen json_gen, char *buffer, int number, const char state = sysctl_rslt; if (state == 0 && present_rate == 100) status = CS_FULL; - else if (state == 0 && present_rate < 100) + else if ((state & ACPI_BATT_STAT_CHARGING) && present_rate < 100) status = CS_CHARGING; else status = CS_DISCHARGING; @@ -248,7 +249,7 @@ void print_battery_info(yajl_gen json_gen, char *buffer, int number, const char (void)snprintf(percentagebuf, sizeof(percentagebuf), "%02d%s", present_rate, pct_mark); - if (state == 1) { + if (state == ACPI_BATT_STAT_DISCHARG) { int hours, minutes; minutes = remaining; hours = minutes / 60;