summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
Diffstat (limited to 'src')
-rw-r--r--src/print_battery_info.c157
1 files changed, 60 insertions, 97 deletions
diff --git a/src/print_battery_info.c b/src/print_battery_info.c
index 7fb2ab5..6b20114 100644
--- a/src/print_battery_info.c
+++ b/src/print_battery_info.c
@@ -38,9 +38,6 @@ void print_battery_info(yajl_gen json_gen, char *buffer, int number, const char
time_t empty_time;
struct tm *empty_tm;
char buf[1024];
- char statusbuf[16];
- char percentagebuf[16];
- char remainingbuf[256];
char emptytimebuf[256];
char consumptionbuf[256];
const char *walk, *last;
@@ -50,12 +47,11 @@ void print_battery_info(yajl_gen json_gen, char *buffer, int number, const char
int full_design = -1,
remaining = -1,
present_rate = -1,
- voltage = -1;
+ voltage = -1,
+ seconds_remaining = -1;
+ float percentage_remaining = -1;
charging_status_t status = CS_DISCHARGING;
- memset(statusbuf, '\0', sizeof(statusbuf));
- memset(percentagebuf, '\0', sizeof(percentagebuf));
- memset(remainingbuf, '\0', sizeof(remainingbuf));
memset(emptytimebuf, '\0', sizeof(emptytimebuf));
memset(consumptionbuf, '\0', sizeof(consumptionbuf));
@@ -63,9 +59,6 @@ void print_battery_info(yajl_gen json_gen, char *buffer, int number, const char
sprintf(batpath, path, number);
INSTANCE(batpath);
-#define BATT_STATUS_NAME(status) \
- (status == CS_CHARGING ? status_chr : (status == CS_DISCHARGING ? status_bat : (status == CS_UNKNOWN ? status_unk : status_full)))
-
#if defined(LINUX)
if (!slurp(batpath, buf, sizeof(buf))) {
OUTPUT_FULL_TEXT(format_down);
@@ -139,9 +132,7 @@ void print_battery_info(yajl_gen json_gen, char *buffer, int number, const char
return;
}
- (void)snprintf(statusbuf, sizeof(statusbuf), "%s", BATT_STATUS_NAME(status));
-
- float percentage_remaining = (((float)remaining / (float)full_design) * 100);
+ percentage_remaining = (((float)remaining / (float)full_design) * 100);
/* Some batteries report POWER_SUPPLY_CHARGE_NOW=<full_design> when fully
* charged, even though that’s plainly wrong. For people who chose to see
* the percentage calculated based on the last full capacity, we clamp the
@@ -150,15 +141,9 @@ void print_battery_info(yajl_gen json_gen, char *buffer, int number, const char
if (last_full_capacity && percentage_remaining > 100) {
percentage_remaining = 100;
}
- if (integer_battery_capacity) {
- (void)snprintf(percentagebuf, sizeof(percentagebuf), "%.00f%s", percentage_remaining, pct_mark);
- } else {
- (void)snprintf(percentagebuf, sizeof(percentagebuf), "%.02f%s", percentage_remaining, pct_mark);
- }
if (present_rate > 0 && status != CS_FULL) {
float remaining_time;
- int seconds, hours, minutes, seconds_remaining;
if (status == CS_CHARGING)
remaining_time = ((float)full_design - (float)remaining) / (float)present_rate;
else if (status == CS_DISCHARGING)
@@ -168,11 +153,6 @@ void print_battery_info(yajl_gen json_gen, char *buffer, int number, const char
seconds_remaining = (int)(remaining_time * 3600.0);
- hours = seconds_remaining / 3600;
- seconds = seconds_remaining - (hours * 3600);
- minutes = seconds / 60;
- seconds -= (minutes * 60);
-
if (status == CS_DISCHARGING && low_threshold > 0) {
if (strcasecmp(threshold_type, "percentage") == 0 && percentage_remaining < low_threshold) {
START_COLOR("color_bad");
@@ -185,13 +165,6 @@ void print_battery_info(yajl_gen json_gen, char *buffer, int number, const char
}
}
- if (hide_seconds)
- (void)snprintf(remainingbuf, sizeof(remainingbuf), "%02d:%02d",
- max(hours, 0), max(minutes, 0));
- else
- (void)snprintf(remainingbuf, sizeof(remainingbuf), "%02d:%02d:%02d",
- max(hours, 0), max(minutes, 0), max(seconds, 0));
-
empty_time = time(NULL);
empty_time += seconds_remaining;
empty_tm = localtime(&empty_time);
@@ -226,44 +199,35 @@ void print_battery_info(yajl_gen json_gen, char *buffer, int number, const char
return;
}
- present_rate = sysctl_rslt;
+ integer_battery_capacity = true;
+ percentage_remaining = sysctl_rslt;
if (sysctlbyname(BATT_TIME, &sysctl_rslt, &sysctl_size, NULL, 0) != 0) {
OUTPUT_FULL_TEXT(format_down);
return;
}
- remaining = sysctl_rslt;
+ hide_seconds = true;
+ seconds_remaining = sysctl_rslt * 60;
if (sysctlbyname(BATT_STATE, &sysctl_rslt, &sysctl_size, NULL, 0) != 0) {
OUTPUT_FULL_TEXT(format_down);
return;
}
state = sysctl_rslt;
- if (state == 0 && present_rate == 100)
+ if (state == 0 && percentage_remaining == 100)
status = CS_FULL;
- else if ((state & ACPI_BATT_STAT_CHARGING) && present_rate < 100)
+ else if ((state & ACPI_BATT_STAT_CHARGING) && percentage_remaining < 100)
status = CS_CHARGING;
else
status = CS_DISCHARGING;
full_design = sysctl_rslt;
- (void)snprintf(statusbuf, sizeof(statusbuf), "%s", BATT_STATUS_NAME(status));
-
- (void)snprintf(percentagebuf, sizeof(percentagebuf), "%02d%s",
- present_rate, pct_mark);
-
if (state == ACPI_BATT_STAT_DISCHARG) {
- int hours, minutes;
- minutes = remaining;
- hours = minutes / 60;
- minutes -= (hours * 60);
- (void)snprintf(remainingbuf, sizeof(remainingbuf), "%02d:%02d",
- max(hours, 0), max(minutes, 0));
- if (strcasecmp(threshold_type, "percentage") == 0 && present_rate < low_threshold) {
+ if (strcasecmp(threshold_type, "percentage") == 0 && percentage_remaining < low_threshold) {
START_COLOR("color_bad");
colorful_output = true;
- } else if (strcasecmp(threshold_type, "time") == 0 && remaining < (u_int)low_threshold) {
+ } else if (strcasecmp(threshold_type, "time") == 0 && seconds_remaining < 60 * low_threshold) {
START_COLOR("color_bad");
colorful_output = true;
}
@@ -307,9 +271,8 @@ void print_battery_info(yajl_gen json_gen, char *buffer, int number, const char
break;
}
- (void)snprintf(statusbuf, sizeof(statusbuf), "%s", BATT_STATUS_NAME(status));
- /* integer_battery_capacity is implied as battery_life is already in whole numbers. */
- (void)snprintf(percentagebuf, sizeof(percentagebuf), "%.00d%s", apm_info.battery_life, pct_mark);
+ 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) {
@@ -323,9 +286,7 @@ void print_battery_info(yajl_gen json_gen, char *buffer, int number, const char
/* Can't give a meaningful value for remaining minutes if we're charging. */
if (status != CS_CHARGING) {
- (void)snprintf(remainingbuf, sizeof(remainingbuf), "%02d:%02d", apm_info.minutes_left / 60, apm_info.minutes_left % 60);
- } else {
- (void)snprintf(remainingbuf, sizeof(remainingbuf), "%s", "(CHR)");
+ seconds_remaining = apm_info.minutes_left * 60;
}
if (colorful_output)
@@ -493,20 +454,9 @@ void print_battery_info(yajl_gen json_gen, char *buffer, int number, const char
full_design = (((float)voltage / 1000.0) * ((float)full_design / 1000.0));
}
- float percentage_remaining =
+ percentage_remaining =
(((float)remaining / (float)full_design) * 100);
- if (integer_battery_capacity)
- (void)snprintf(percentagebuf,
- sizeof(percentagebuf),
- "%d%s",
- (int)percentage_remaining, pct_mark);
- else
- (void)snprintf(percentagebuf,
- sizeof(percentagebuf),
- "%.02f%s",
- percentage_remaining, pct_mark);
-
/*
* Handle percentage low_threshold here, and time low_threshold when
* we have it.
@@ -519,16 +469,13 @@ void print_battery_info(yajl_gen json_gen, char *buffer, int number, const char
}
if (is_full)
- (void)snprintf(statusbuf, sizeof(statusbuf), "%s", BATT_STATUS_NAME(CS_FULL));
- else
- (void)snprintf(statusbuf, sizeof(statusbuf), "%s", BATT_STATUS_NAME(status));
+ status = CS_FULL;
/*
* The envsys(4) ACPI routines do not appear to provide a 'time
* remaining' figure, so we must deduce it.
*/
float remaining_time;
- int seconds, hours, minutes, seconds_remaining;
if (status == CS_CHARGING)
remaining_time = ((float)full_design - (float)remaining) / (float)present_rate;
@@ -539,32 +486,13 @@ void print_battery_info(yajl_gen json_gen, char *buffer, int number, const char
seconds_remaining = (int)(remaining_time * 3600.0);
- hours = seconds_remaining / 3600;
- seconds = seconds_remaining - (hours * 3600);
- minutes = seconds / 60;
- seconds -= (minutes * 60);
-
if (status != CS_CHARGING) {
- if (hide_seconds)
- (void)snprintf(remainingbuf, sizeof(remainingbuf), "%02d:%02d",
- max(hours, 0), max(minutes, 0));
- else
- (void)snprintf(remainingbuf, sizeof(remainingbuf), "%02d:%02d:%02d",
- max(hours, 0), max(minutes, 0), max(seconds, 0));
-
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;
}
}
- } else {
- if (hide_seconds)
- (void)snprintf(remainingbuf, sizeof(remainingbuf), "(%02d:%02d until full)",
- max(hours, 0), max(minutes, 0));
- else
- (void)snprintf(remainingbuf, sizeof(remainingbuf), "(%02d:%02d:%02d until full)",
- max(hours, 0), max(minutes, 0), max(seconds, 0));
}
empty_time = time(NULL);
@@ -585,9 +513,9 @@ void print_battery_info(yajl_gen json_gen, char *buffer, int number, const char
((float)present_rate / 1000.0 / 1000.0));
#endif
-#define EAT_SPACE_FROM_OUTPUT_IF_EMPTY(_buf) \
+#define EAT_SPACE_FROM_OUTPUT_IF_NO_OUTPUT() \
do { \
- if (strlen(_buf) == 0) { \
+ if (outwalk == prevoutwalk) { \
if (outwalk > buffer && isspace(outwalk[-1])) \
outwalk--; \
else if (isspace(*(walk + 1))) \
@@ -596,29 +524,64 @@ void print_battery_info(yajl_gen json_gen, char *buffer, int number, const char
} while (0)
for (walk = format; *walk != '\0'; walk++) {
+ char *prevoutwalk = outwalk;
+
if (*walk != '%') {
*(outwalk++) = *walk;
continue;
}
if (BEGINS_WITH(walk + 1, "status")) {
- outwalk += sprintf(outwalk, "%s", statusbuf);
+ const char *statusstr;
+ switch (status) {
+ case CS_CHARGING:
+ statusstr = status_chr;
+ break;
+ case CS_DISCHARGING:
+ statusstr = status_bat;
+ break;
+ case CS_FULL:
+ statusstr = status_full;
+ break;
+ default:
+ statusstr = status_unk;
+ }
+
+ outwalk += sprintf(outwalk, "%s", statusstr);
walk += strlen("status");
} else if (BEGINS_WITH(walk + 1, "percentage")) {
- outwalk += sprintf(outwalk, "%s", percentagebuf);
+ if (integer_battery_capacity) {
+ outwalk += sprintf(outwalk, "%.00f%s", percentage_remaining, pct_mark);
+ } else {
+ outwalk += sprintf(outwalk, "%.02f%s", percentage_remaining, pct_mark);
+ }
walk += strlen("percentage");
} else if (BEGINS_WITH(walk + 1, "remaining")) {
- outwalk += sprintf(outwalk, "%s", remainingbuf);
+ if (seconds_remaining >= 0) {
+ int seconds, hours, minutes;
+
+ hours = seconds_remaining / 3600;
+ seconds = seconds_remaining - (hours * 3600);
+ minutes = seconds / 60;
+ seconds -= (minutes * 60);
+
+ if (hide_seconds)
+ outwalk += sprintf(outwalk, "%02d:%02d",
+ max(hours, 0), max(minutes, 0));
+ else
+ outwalk += sprintf(outwalk, "%02d:%02d:%02d",
+ max(hours, 0), max(minutes, 0), max(seconds, 0));
+ }
walk += strlen("remaining");
- EAT_SPACE_FROM_OUTPUT_IF_EMPTY(remainingbuf);
+ EAT_SPACE_FROM_OUTPUT_IF_NO_OUTPUT();
} else if (BEGINS_WITH(walk + 1, "emptytime")) {
outwalk += sprintf(outwalk, "%s", emptytimebuf);
walk += strlen("emptytime");
- EAT_SPACE_FROM_OUTPUT_IF_EMPTY(emptytimebuf);
+ EAT_SPACE_FROM_OUTPUT_IF_NO_OUTPUT();
} else if (BEGINS_WITH(walk + 1, "consumption")) {
outwalk += sprintf(outwalk, "%s", consumptionbuf);
walk += strlen("consumption");
- EAT_SPACE_FROM_OUTPUT_IF_EMPTY(consumptionbuf);
+ EAT_SPACE_FROM_OUTPUT_IF_NO_OUTPUT();
}
}
generated by cgit v1.2.3 (git 2.39.1) at 2024-12-26 06:18:10 +0000