From 6d3e9991de8f3299be07c1099e03b5d0568048ad Mon Sep 17 00:00:00 2001 From: Tommie Gannert Date: Mon, 1 Aug 2016 01:15:35 +0100 Subject: Move START_COLOR to after OS-specific code. This uses the more direct tests of whether percentage_remaining and seconds_remaining are available or not (rather than looking at present_rate). --- src/print_battery_info.c | 71 +++++++----------------------------------------- 1 file changed, 10 insertions(+), 61 deletions(-) diff --git a/src/print_battery_info.c b/src/print_battery_info.c index 09db0a2..9afcca9 100644 --- a/src/print_battery_info.c +++ b/src/print_battery_info.c @@ -39,7 +39,6 @@ void print_battery_info(yajl_gen json_gen, char *buffer, int number, const char const char *walk, *last; char *outwalk = buffer; bool watt_as_unit = false; - bool colorful_output = false; int full_design = -1, remaining = -1, present_rate = -1, @@ -145,28 +144,6 @@ void print_battery_info(yajl_gen json_gen, char *buffer, int number, const char remaining_time = 0; seconds_remaining = (int)(remaining_time * 3600.0); - - if (status == CS_DISCHARGING && low_threshold > 0) { - if (strcasecmp(threshold_type, "percentage") == 0 && percentage_remaining < low_threshold) { - START_COLOR("color_bad"); - colorful_output = true; - } else if (strcasecmp(threshold_type, "time") == 0 && seconds_remaining < 60 * low_threshold) { - START_COLOR("color_bad"); - colorful_output = true; - } else { - colorful_output = false; - } - } - } else { - /* On some systems, present_rate may not exist. Still, make sure - * we colorize the output if threshold_type is set to percentage - * (since we don't have any information on remaining time). */ - if (status == CS_DISCHARGING && low_threshold > 0) { - if (strcasecmp(threshold_type, "percentage") == 0 && percentage_remaining < low_threshold) { - START_COLOR("color_bad"); - colorful_output = true; - } - } } #elif defined(__FreeBSD__) || defined(__FreeBSD_kernel__) || defined(__DragonFly__) int state; @@ -201,16 +178,6 @@ void print_battery_info(yajl_gen json_gen, char *buffer, int number, const char status = CS_DISCHARGING; full_design = sysctl_rslt; - - if (state == ACPI_BATT_STAT_DISCHARG) { - if (strcasecmp(threshold_type, "percentage") == 0 && percentage_remaining < low_threshold) { - START_COLOR("color_bad"); - colorful_output = true; - } else if (strcasecmp(threshold_type, "time") == 0 && seconds_remaining < 60 * low_threshold) { - START_COLOR("color_bad"); - colorful_output = true; - } - } #elif defined(__OpenBSD__) /* * We're using apm(4) here, which is the interface to acpi(4) on amd64/i386 and @@ -253,16 +220,6 @@ void print_battery_info(yajl_gen json_gen, char *buffer, int number, const char integer_battery_capacity = true; percentage_remaining = apm_info.battery_life; - if (status == CS_DISCHARGING && low_threshold > 0) { - if (strcasecmp(threshold_type, "percentage") == 0 && apm_info.battery_life < low_threshold) { - START_COLOR("color_bad"); - colorful_output = true; - } else if (strcasecmp(threshold_type, "time") == 0 && apm_info.minutes_left < (u_int)low_threshold) { - START_COLOR("color_bad"); - colorful_output = true; - } - } - /* Can't give a meaningful value for remaining minutes if we're charging. */ if (status != CS_CHARGING) { seconds_remaining = apm_info.minutes_left * 60; @@ -436,17 +393,6 @@ void print_battery_info(yajl_gen json_gen, char *buffer, int number, const char percentage_remaining = (((float)remaining / (float)full_design) * 100); - /* - * Handle percentage low_threshold here, and time low_threshold when - * we have it. - */ - if (status == CS_DISCHARGING && low_threshold > 0) { - if (strcasecmp(threshold_type, "percentage") == 0 && (((float)remaining / (float)full_design) * 100) < low_threshold) { - START_COLOR("color_bad"); - colorful_output = true; - } - } - if (is_full) status = CS_FULL; @@ -464,16 +410,19 @@ void print_battery_info(yajl_gen json_gen, char *buffer, int number, const char remaining_time = 0; seconds_remaining = (int)(remaining_time * 3600.0); +#endif - if (status != CS_CHARGING) { - if (low_threshold > 0) { - if (strcasecmp(threshold_type, "time") == 0 && ((float)seconds_remaining / 60.0) < (u_int)low_threshold) { - START_COLOR("color_bad"); - colorful_output = true; - } + bool colorful_output = false; + + if (status == CS_DISCHARGING && low_threshold > 0) { + if (percentage_remaining >= 0 && strcasecmp(threshold_type, "percentage") == 0 && percentage_remaining < low_threshold) { + START_COLOR("color_bad"); + colorful_output = true; + } else if (seconds_remaining >= 0 && strcasecmp(threshold_type, "time") == 0 && seconds_remaining < 60 * low_threshold) { + START_COLOR("color_bad"); + colorful_output = true; } } -#endif #define EAT_SPACE_FROM_OUTPUT_IF_NO_OUTPUT() \ do { \ -- cgit v1.2.3 From 52f0dd6a36099d238796eccd16410be79325c806 Mon Sep 17 00:00:00 2001 From: Tommie Gannert Date: Mon, 1 Aug 2016 01:19:23 +0100 Subject: Remove an END_COLOR in print_battery_info for OpenBSD. This was probably a bug since it makes no sense to have here. --- src/print_battery_info.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/src/print_battery_info.c b/src/print_battery_info.c index 9afcca9..d0e69c4 100644 --- a/src/print_battery_info.c +++ b/src/print_battery_info.c @@ -224,9 +224,6 @@ void print_battery_info(yajl_gen json_gen, char *buffer, int number, const char if (status != CS_CHARGING) { seconds_remaining = apm_info.minutes_left * 60; } - - if (colorful_output) - END_COLOR; #elif defined(__NetBSD__) /* * Using envsys(4) via sysmon(4). -- cgit v1.2.3 From 0b16860145e0363f1806ecbb94c92b08c51e1481 Mon Sep 17 00:00:00 2001 From: Tommie Gannert Date: Mon, 1 Aug 2016 01:27:44 +0100 Subject: Refactor common seconds_remaining code in print_battery_info. --- src/print_battery_info.c | 35 ++++++++++++++++------------------- 1 file changed, 16 insertions(+), 19 deletions(-) diff --git a/src/print_battery_info.c b/src/print_battery_info.c index d0e69c4..fc75f52 100644 --- a/src/print_battery_info.c +++ b/src/print_battery_info.c @@ -28,6 +28,20 @@ #include #endif +/* + * Estimate the number of seconds remaining in state 'status'. + * + * Assumes a constant (dis)charge rate. + */ +static int seconds_remaining_from_rate(charging_status_t status, float full_design, float remaining, float present_rate) { + if (status == CS_CHARGING) + return 3600.0 * (full_design - remaining) / present_rate; + else if (status == CS_DISCHARGING) + return 3600.0 * remaining / present_rate; + else + return 0; +} + /* * Get battery information from /sys. Note that it uses the design capacity to * calculate the percentage, not the last full capacity, so you can see how @@ -135,15 +149,7 @@ void print_battery_info(yajl_gen json_gen, char *buffer, int number, const char } if (present_rate > 0 && status != CS_FULL) { - float remaining_time; - if (status == CS_CHARGING) - remaining_time = ((float)full_design - (float)remaining) / (float)present_rate; - else if (status == CS_DISCHARGING) - remaining_time = ((float)remaining / (float)present_rate); - else - remaining_time = 0; - - seconds_remaining = (int)(remaining_time * 3600.0); + seconds_remaining = seconds_remaining_from_rate(status, full_design, remaining, present_rate); } #elif defined(__FreeBSD__) || defined(__FreeBSD_kernel__) || defined(__DragonFly__) int state; @@ -397,16 +403,7 @@ void print_battery_info(yajl_gen json_gen, char *buffer, int number, const char * The envsys(4) ACPI routines do not appear to provide a 'time * remaining' figure, so we must deduce it. */ - float remaining_time; - - if (status == CS_CHARGING) - remaining_time = ((float)full_design - (float)remaining) / (float)present_rate; - else if (status == CS_DISCHARGING) - remaining_time = ((float)remaining / (float)present_rate); - else - remaining_time = 0; - - seconds_remaining = (int)(remaining_time * 3600.0); + seconds_remaining = seconds_remaining_from_rate(status, full_design, remaining, present_rate); #endif bool colorful_output = false; -- cgit v1.2.3