diff options
| author | drkhsh <me@drkhsh.at> | 2022-10-28 00:15:21 +0200 |
|---|---|---|
| committer | drkhsh <me@drkhsh.at> | 2022-10-28 01:02:58 +0200 |
| commit | 21327e0373189f9e9964f34fb5a15639d00575aa (patch) | |
| tree | eceba7815e48b8f6e88f7330483ee0d96d08f65f | |
| parent | 69b2487650782f135db76078c4a7fb841cb936ac (diff) | |
| download | slstatus-21327e0373189f9e9964f34fb5a15639d00575aa.tar.gz | |
temp: Put sysctl into define to avoid line wraps
Long, wrapped, multi-line if statements suck to read.
This fixes readability.
| -rw-r--r-- | components/temperature.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/components/temperature.c b/components/temperature.c index 306a09f..01d21b9 100644 --- a/components/temperature.c +++ b/components/temperature.c @@ -53,6 +53,8 @@ #include <stdlib.h> #include <sys/sysctl.h> + #define ACPI_TEMP "hw.acpi.thermal.%s.temperature" + const char * temp(const char *zone) { @@ -61,8 +63,8 @@ size_t len; len = sizeof(temp); - snprintf(buf, sizeof(buf), "hw.acpi.thermal.%s.temperature", zone); - if (sysctlbyname(buf, &temp, &len, NULL, 0) == -1 + snprintf(buf, sizeof(buf), ACPI_TEMP, zone); + if (sysctlbyname(buf, &temp, &len, NULL, 0) < 0 || !len) return NULL; |