diff options
author | Aaron Marcher <me@drkhsh.at> | 2018-05-06 22:28:56 +0200 |
---|---|---|
committer | Aaron Marcher <me@drkhsh.at> | 2018-05-06 22:28:56 +0200 |
commit | ee5ec756218c852385c5ba5ef0f75b47ba59ec39 (patch) | |
tree | 2cc3cd4e654671d509dceb572c7c72d3bfde5e9d /components/cpu.c | |
parent | 66a7fb16fc903ac7ef6ce1acdbb353f1072cd0ec (diff) |
Fix coding style
- Use block for single statement ifs
- Keep lines to reasonable length (current debate as to reasonable)
- When functions return -1 for error test against 0 not -1
- Do not indent cases another level
- Do not test against NULL and 0 explicitly
- Use tabs for indentation, use spaces for alignment
Diffstat (limited to 'components/cpu.c')
-rw-r--r-- | components/cpu.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/components/cpu.c b/components/cpu.c index c8299f9..3b29b8b 100644 --- a/components/cpu.c +++ b/components/cpu.c @@ -58,9 +58,8 @@ size = sizeof(freq); - if (sysctl(mib, 2, &freq, &size, NULL, 0) == -1) { - fprintf(stderr, "sysctl 'HW_CPUSPEED': %s\n", - strerror(errno)); + if (sysctl(mib, 2, &freq, &size, NULL, 0) < 0) { + fprintf(stderr, "sysctl 'HW_CPUSPEED': %s\n", strerror(errno)); return NULL; } @@ -82,7 +81,7 @@ size = sizeof(a); memcpy(b, a, sizeof(b)); - if (sysctl(mib, 2, &a, &size, NULL, 0) == -1) { + if (sysctl(mib, 2, &a, &size, NULL, 0) < 0) { fprintf(stderr, "sysctl 'KERN_CPTIME': %s\n", strerror(errno)); return NULL; } |