Add a new header defining commonly-used exit codes.

"Commonly-used" meaning "used by more than one source file".

Clean up the exit codes, combining some duplicates with different names,
and using some instead of raw numbers in some places.

1 files changed, 2 insertions, 4 deletions

diff --git a/capinfos.c b/capinfos.c
index 00b2e74c5d..4cfa45510b 100644
--- a/capinfos.c
+++ b/capinfos.c
@@ -69,6 +69,7 @@
 #include <wiretap/wtap.h>
 
 #include <ui/cmdarg_err.h>
+#include <ui/exit_codes.h>
 #include <wsutil/filesystem.h>
 #include <wsutil/privileges.h>
 #include <cli_main.h>
@@ -87,9 +88,6 @@
 
 #include "ui/failure_message.h"
 
-#define INVALID_OPTION 1
-#define BAD_FLAG 1
-
 /*
  * By default capinfos now continues processing
  * the next filename if and when wiretap detects
@@ -1792,7 +1790,7 @@ main(int argc, char *argv[])
 
       case '?':              /* Bad flag - print usage message */
         print_usage(stderr);
-        overall_error_status = BAD_FLAG;
+        overall_error_status = INVALID_OPTION;
         goto exit;
         break;
     }