From: John Baldwin <jhb@FreeBSD.org>
Date: Thu, 8 Oct 2015 20:15:38 +0000 (-0700)
Subject: Properly detect the battery charging status under FreeBSD.
X-Git-Tag: 2.10~9^2
X-Git-Url: https://git.sur5r.net/?p=i3%2Fi3status;a=commitdiff_plain;h=5e33d9fe748efc8df49c123fe01d486b04c20f57;hp=974f95702efbf1ae15882777d191c7fde99bfb9b

Properly detect the battery charging status under FreeBSD.

The hw.acpi.battery.state sysctl returns a bitmask of flags as
defined in <dev/acpica/acpiio.h>.  Use constants from this header
to examine the state and check for the charging flag to determine
if the battery is charging.
---

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 <sys/types.h>
 #include <sys/sysctl.h>
+#include <dev/acpica/acpiio.h>
 #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;