[hackers] [slstatus] radical re-formatting 3/3: Error checks || drkhsh

[git]

commit cce2e5ecb05cdf68831fbf44c5ab90f4e16364b3
Author:     drkhsh <m...@drkhsh.at>
AuthorDate: Fri Oct 28 00:21:02 2022 +0200
Commit:     drkhsh <m...@drkhsh.at>
CommitDate: Fri Oct 28 01:03:46 2022 +0200

    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/

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;
                }
diff --git a/components/netspeeds.c b/components/netspeeds.c
index 97e565b..cde6fa9 100644
--- a/components/netspeeds.c
+++ b/components/netspeeds.c
@@ -71,7 +71,7 @@
 
                oldrxbytes = rxbytes;
 
-               if (getifaddrs(&ifal) == -1) {
+               if (getifaddrs(&ifal) < 0) {
                        warn("getifaddrs failed");
                        return NULL;
                }
@@ -105,7 +105,7 @@
 
                oldtxbytes = txbytes;
 
-               if (getifaddrs(&ifal) == -1) {
+               if (getifaddrs(&ifal) < 0) {
                        warn("getifaddrs failed");
                        return NULL;
                }
diff --git a/components/ram.c b/components/ram.c
index 326153f..15c4b74 100644
--- a/components/ram.c
+++ b/components/ram.c
@@ -159,8 +159,8 @@
                size_t len;
 
                len = sizeof(struct vmtotal);
-               if (sysctl(mib, 2, &vm_stats, &len, NULL, 0) == -1
-                               || !len)
+               if (sysctl(mib, 2, &vm_stats, &len, NULL, 0) < 0
+                   || !len)
                        return NULL;
 
                return fmt_human(vm_stats.t_free * getpagesize(), 1024);
@@ -172,8 +172,8 @@
                size_t len;
 
                len = sizeof(npages);
-               if (sysctlbyname("vm.stats.vm.v_page_count", &npages, &len, 
NULL, 0) == -1
-                               || !len)
+               if (sysctlbyname("vm.stats.vm.v_page_count",
+                                &npages, &len, NULL, 0) < 0 || !len)
                        return NULL;
 
                return fmt_human(npages * getpagesize(), 1024);
@@ -186,12 +186,12 @@
                size_t len;
 
                len = sizeof(npages);
-               if (sysctlbyname("vm.stats.vm.v_page_count", &npages, &len, 
NULL, 0) == -1
-                               || !len)
+               if (sysctlbyname("vm.stats.vm.v_page_count",
+                                &npages, &len, NULL, 0) < 0 || !len)
                        return NULL;
 
-               if (sysctlbyname("vm.stats.vm.v_active_count", &active, &len, 
NULL, 0) == -1
-                               || !len)
+               if (sysctlbyname("vm.stats.vm.v_active_count",
+                                &active, &len, NULL, 0) < 0 || !len)
                        return NULL;
 
                return bprintf("%d", active * 100 / npages);
@@ -203,8 +203,8 @@
                size_t len;
 
                len = sizeof(active);
-               if (sysctlbyname("vm.stats.vm.v_active_count", &active, &len, 
NULL, 0) == -1
-                               || !len)
+               if (sysctlbyname("vm.stats.vm.v_active_count",
+                                &active, &len, NULL, 0) < 0 || !len)
                        return NULL;
 
                return fmt_human(active * getpagesize(), 1024);
diff --git a/components/swap.c b/components/swap.c
index 6ae0542..f270d93 100644
--- a/components/swap.c
+++ b/components/swap.c
@@ -204,7 +204,7 @@
                        return 0;
                }
 
-               if(kvm_getswapinfo(kd, swap_info, size, 0 /* Unused flags */) 
== -1) {
+               if (kvm_getswapinfo(kd, swap_info, size, 0 /* Unused flags */) 
< 0) {
                        warn("kvm_getswapinfo:");
                        kvm_close(kd);
                        return 0;