diff options
author | Jim Meyering <jim@meyering.net> | 2004-01-21 22:50:28 +0000 |
---|---|---|
committer | Jim Meyering <jim@meyering.net> | 2004-01-21 22:50:28 +0000 |
commit | e0261d2e7271a14b767e6851240881f4a0ef4648 (patch) | |
tree | 99b47aef63bc0cbc09a9a85f3b24380a65413f45 | |
parent | a8fd56864ef1c93ca87094a3eecebc06566d6ca0 (diff) | |
download | coreutils-e0261d2e7271a14b767e6851240881f4a0ef4648.tar.xz |
(usage): Use EXIT_SUCCESS, not 0, for clarity.
(usage): Don't bother normalizing exit status
since the arg is already the correct exit status now.
(FATAL_ERROR, main): Exit with status EXIT_FAILURE, not 2, on errors.
-rw-r--r-- | src/cut.c | 11 |
1 files changed, 6 insertions, 5 deletions
@@ -1,5 +1,6 @@ /* cut - remove parts of lines of files - Copyright (C) 1984, 1997-2003 by David M. Ihnat + Copyright (C) 1997-2004 Free Software Foundation, Inc. + Copyright (C) 1984 David M. Ihnat This program is free software; you can redistribute it and/or modify it under the terms of the GNU General Public License as published by @@ -45,7 +46,7 @@ do \ { \ error (0, 0, (Message)); \ - usage (2); \ + usage (EXIT_FAILURE); \ } \ while (0) @@ -173,7 +174,7 @@ static struct option const longopts[] = void usage (int status) { - if (status != 0) + if (status != EXIT_SUCCESS) fprintf (stderr, _("Try `%s --help' for more information.\n"), program_name); else @@ -221,7 +222,7 @@ With no FILE, or when FILE is -, read standard input.\n\ "), stdout); printf (_("\nReport bugs to <%s>.\n"), PACKAGE_BUGREPORT); } - exit (status == 0 ? EXIT_SUCCESS : EXIT_FAILURE); + exit (status); } static inline void @@ -794,7 +795,7 @@ main (int argc, char **argv) case_GETOPT_VERSION_CHAR (PROGRAM_NAME, AUTHORS); default: - usage (2); + usage (EXIT_FAILURE); } } |