From ee5ec756218c852385c5ba5ef0f75b47ba59ec39 Mon Sep 17 00:00:00 2001 From: Aaron Marcher Date: Sun, 6 May 2018 22:28:56 +0200 Subject: 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 --- components/cpu.c | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) (limited to 'components/cpu.c') 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; } -- cgit v1.2.3