diff options
| author | drkhsh <me@drkhsh.at> | 2022-10-28 00:21:02 +0200 |
|---|---|---|
| committer | drkhsh <me@drkhsh.at> | 2022-10-28 01:03:46 +0200 |
| commit | cce2e5ecb05cdf68831fbf44c5ab90f4e16364b3 (patch) | |
| tree | 7caa1faa12adf6f32dfaf10a61cc69985b9bcb79 /components/cpu.c | |
| parent | 3251e911878b78b0aad8fc09c782f8d81c878f75 (diff) | |
| download | slstatus-cce2e5ecb05cdf68831fbf44c5ab90f4e16364b3.tar.gz | |
radical re-formatting 3/3: Error checks
Check for `< 0` instead of `== -1`.
Fixes coding style. Formatting commits suck, incoherent coding style
sucks more.
https://suckless.org/coding_style/
Diffstat (limited to 'components/cpu.c')
| -rw-r--r-- | components/cpu.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/components/cpu.c b/components/cpu.c index 10d25c0..d0d03c7 100644 --- a/components/cpu.c +++ b/components/cpu.c @@ -118,8 +118,7 @@ size = sizeof(freq); /* in MHz */ - if (sysctlbyname("hw.clockrate", &freq, &size, NULL, 0) == -1 - || !size) { + if (sysctlbyname("hw.clockrate", &freq, &size, NULL, 0) < 0 || !size) { warn("sysctlbyname 'hw.clockrate':"); return NULL; } @@ -136,8 +135,7 @@ size = sizeof(a); memcpy(b, a, sizeof(b)); - if (sysctlbyname("kern.cp_time", &a, &size, NULL, 0) == -1 - || !size) { + if (sysctlbyname("kern.cp_time", &a, &size, NULL, 0) < 0 || !size) { warn("sysctlbyname 'kern.cp_time':"); return NULL; } |