diff options
| author | drkhsh <me@drkhsh.at> | 2022-10-27 23:44:52 +0200 |
|---|---|---|
| committer | drkhsh <me@drkhsh.at> | 2022-10-28 01:01:10 +0200 |
| commit | 69b2487650782f135db76078c4a7fb841cb936ac (patch) | |
| tree | e236b6b8503587b9263f45e775204a64463c5ca3 /components/netspeeds.c | |
| parent | c46c1487a986496dd813ec52e17e5bf9ba10fd84 (diff) | |
| download | slstatus-69b2487650782f135db76078c4a7fb841cb936ac.tar.gz | |
various: Put paths into defines to avoid line wraps
Long, wrapped, multi-line if statements suck to read.
This fixes readability in the worst places by packing format strings for
paths into defines.
Diffstat (limited to 'components/netspeeds.c')
| -rw-r--r-- | components/netspeeds.c | 13 |
1 files changed, 5 insertions, 8 deletions
diff --git a/components/netspeeds.c b/components/netspeeds.c index f7bac2e..3a1817b 100644 --- a/components/netspeeds.c +++ b/components/netspeeds.c @@ -8,6 +8,9 @@ #if defined(__linux__) #include <stdint.h> + #define NET_RX_BYTES "/sys/class/net/%s/statistics/rx_bytes" + #define NET_TX_BYTES "/sys/class/net/%s/statistics/tx_bytes" + const char * netspeed_rx(const char *interface) { @@ -18,11 +21,8 @@ oldrxbytes = rxbytes; - if (esnprintf(path, sizeof(path), - "/sys/class/net/%s/statistics/rx_bytes", - interface) < 0) { + if (esnprintf(path, sizeof(path), NET_RX_BYTES, interface) < 0) return NULL; - } if (pscanf(path, "%ju", &rxbytes) != 1) { return NULL; } @@ -44,11 +44,8 @@ oldtxbytes = txbytes; - if (esnprintf(path, sizeof(path), - "/sys/class/net/%s/statistics/tx_bytes", - interface) < 0) { + if (esnprintf(path, sizeof(path), NET_TX_BYTES, interface) < 0) return NULL; - } if (pscanf(path, "%ju", &txbytes) != 1) { return NULL; } |