diff options
| author | tb <tb@cvs.openbsd.org> | 2015-12-24 15:01:25 +0000 |
|---|---|---|
| committer | tb <tb@cvs.openbsd.org> | 2015-12-24 15:01:25 +0000 |
| commit | ceb06b644c9c0961b749a529b82c7d026323e57e (patch) | |
| tree | be984cc061005b13d0e776860fbaded0606d4207 /usr.bin | |
| parent | 3f9edfcd6c16fa9ee76196cb366ba73ec87777ee (diff) | |
KNF cleanup: sort #includes and cases in switch statements.
Fix whitespace issues with if-statements.
Use return instead of exit.
From fritjof () alokat ! org, thanks!
ok benno@
Diffstat (limited to 'usr.bin')
| -rw-r--r-- | usr.bin/uname/uname.c | 42 |
1 files changed, 25 insertions, 17 deletions
diff --git a/usr.bin/uname/uname.c b/usr.bin/uname/uname.c index ad8dcb4a76c..97d84a65412 100644 --- a/usr.bin/uname/uname.c +++ b/usr.bin/uname/uname.c @@ -1,4 +1,4 @@ -/* $OpenBSD: uname.c,v 1.16 2015/10/09 01:37:09 deraadt Exp $ */ +/* $OpenBSD: uname.c,v 1.17 2015/12/24 15:01:24 tb Exp $ */ /* * Copyright (c) 1994 Winning Strategies, Inc. @@ -32,12 +32,13 @@ */ #include <sys/param.h> /* MACHINE_ARCH */ +#include <sys/utsname.h> + +#include <err.h> +#include <locale.h> #include <stdio.h> #include <stdlib.h> -#include <locale.h> #include <unistd.h> -#include <sys/utsname.h> -#include <err.h> static void usage(void); @@ -73,6 +74,9 @@ main(int argc, char *argv[]) case 'n': print_mask |= PRINT_NODENAME; break; + case 'p': + print_mask |= PRINT_MACHINE_ARCH; + break; case 'r': print_mask |= PRINT_RELEASE; break; @@ -82,9 +86,6 @@ main(int argc, char *argv[]) case 'v': print_mask |= PRINT_VERSION; break; - case 'p': - print_mask |= PRINT_MACHINE_ARCH; - break; default: usage(); /* NOTREACHED */ @@ -100,39 +101,46 @@ main(int argc, char *argv[]) print_mask = PRINT_SYSNAME; } - if (uname(&u)) { + if (uname(&u)) err(1, NULL); - /* NOTREACHED */ - } if (print_mask & PRINT_SYSNAME) { space++; fputs(u.sysname, stdout); } if (print_mask & PRINT_NODENAME) { - if (space++) putchar(' '); + if (space++) + putchar(' '); + fputs(u.nodename, stdout); } if (print_mask & PRINT_RELEASE) { - if (space++) putchar(' '); + if (space++) + putchar(' '); + fputs(u.release, stdout); } if (print_mask & PRINT_VERSION) { - if (space++) putchar(' '); + if (space++) + putchar(' '); + fputs(u.version, stdout); } if (print_mask & PRINT_MACHINE) { - if (space++) putchar(' '); + if (space++) + putchar(' '); + fputs(u.machine, stdout); } if (print_mask & PRINT_MACHINE_ARCH) { - if (space++) putchar(' '); + if (space++) + putchar(' '); + fputs(MACHINE_ARCH, stdout); } putchar('\n'); - exit(0); - /* NOTREACHED */ + return 0; } static void |