change the hardcoded error constant (0x80000000UL) to a nice flexible define (BB_GETO...

diff --git a/archival/ar.c b/archival/ar.c
index bbdf2fddc20a5c1210b2c5fa60f8189aa80896f5..8326aa6de6aa1a8d07aa0c0397665a2a23003ccb 100644 (file)
--- a/archival/ar.c
+++ b/archival/ar.c
@@ -70,7 +70,7 @@ extern int ar_main(int argc, char **argv)
        bb_opt_complementaly = "p~tx:t~px:x~pt";
        opt = bb_getopt_ulflags(argc, argv, "ptxovcr");
 
-       if ((opt & 0x80000000UL) || (opt == 0) || (optind == argc)) {
+       if ((opt & BB_GETOPT_ERROR) || (opt == 0) || (optind == argc)) {
                bb_show_usage();
        }
 

diff --git a/archival/dpkg_deb.c b/archival/dpkg_deb.c
index 5aa9881d58dca9306c0753d10726d331522b9e9e..b95ec2d6ef755dc9a56c845c61a9bc369a2716ab 100644 (file)
--- a/archival/dpkg_deb.c
+++ b/archival/dpkg_deb.c
@@ -88,7 +88,7 @@ extern int dpkg_deb_main(int argc, char **argv)
                argcount = 2;
        }
 
-       if ((optind + argcount != argc) || (opt & 0x80000000UL)) {
+       if ((optind + argcount != argc) || (opt & BB_GETOPT_ERROR)) {
                bb_show_usage();
        }
 

diff --git a/archival/tar.c b/archival/tar.c
index 950e21dd3a8298a7bd20585d0b313ed63355b21b..b2a212397d7324f0aa90632261f66a7add5f471b 100644 (file)
--- a/archival/tar.c
+++ b/archival/tar.c
@@ -724,7 +724,7 @@ int tar_main(int argc, char **argv)
                                );
 
        /* Check one and only one context option was given */
-       if(opt & 0x80000000UL) {
+       if(opt & BB_GETOPT_ERROR) {
                bb_show_usage();
        }
 #ifdef CONFIG_FEATURE_TAR_CREATE

diff --git a/coreutils/cut.c b/coreutils/cut.c
index d26e80eee4420d51d33bc481d2c87b187e28657e..e5fb5aff720e222495c52cd7f913216b039430e8 100644 (file)
--- a/coreutils/cut.c
+++ b/coreutils/cut.c
@@ -300,7 +300,7 @@ extern int cut_main(int argc, char **argv)
        part = opt & (OPT_BYTE_FLGS|OPT_CHAR_FLGS|OPT_FIELDS_FLGS);
        if(part == 0)
                bb_error_msg_and_die("you must specify a list of bytes, characters, or fields");
-       if(opt & 0x80000000UL)
+       if(opt & BB_GETOPT_ERROR)
                bb_error_msg_and_die("only one type of list may be specified");
        parse_lists(sopt);
        if((opt & (OPT_DELIM_FLGS))) {

diff --git a/coreutils/date.c b/coreutils/date.c
index 3608df69fceceed08e6967ef16bd97c659e0e8b1..513aadeec9be0e2e37814d0f961eb8470916d70c 100644 (file)
--- a/coreutils/date.c
+++ b/coreutils/date.c
@@ -166,7 +166,7 @@ int date_main(int argc, char **argv)
                bb_error_msg_and_die(bb_msg_memory_exhausted);
        }
        use_arg = opt & DATE_OPT_DATE;
-       if(opt & 0x80000000UL)
+       if(opt & BB_GETOPT_ERROR)
                bb_show_usage();
 #ifdef CONFIG_FEATURE_DATE_ISOFMT
        if(opt & DATE_OPT_TIMESPEC) {

diff --git a/coreutils/id.c b/coreutils/id.c
index 541c3d16b441c9344b0048e777cfa60299ad04b9..6360491688e46d7d90a84d5b511ff1c7cc55ed1d 100644 (file)
--- a/coreutils/id.c
+++ b/coreutils/id.c
@@ -64,7 +64,7 @@ extern int id_main(int argc, char **argv)
        bb_opt_complementaly = "u~g:g~u";
        flags = bb_getopt_ulflags(argc, argv, "rnug");
 
-       if ((flags & 0x80000000UL)
+       if ((flags & BB_GETOPT_ERROR)
        /* Don't allow -n -r -nr */
        || (flags <= 3 && flags > 0) 
        /* Don't allow more than one username */

diff --git a/coreutils/install.c b/coreutils/install.c
index 36dc1d61820978c6214a0187c9c46eac8e807fca..345e75af0943f060d9fb4cff40ecb8d9928e8aad 100644 (file)
--- a/coreutils/install.c
+++ b/coreutils/install.c
@@ -69,7 +69,7 @@ extern int install_main(int argc, char **argv)
        flags = bb_getopt_ulflags(argc, argv, "cdpsg:m:o:", &gid_str, &mode_str, &uid_str);     /* 'a' must be 2nd */
 
        /* Check valid options were given */
-       if(flags & 0x80000000UL) {
+       if(flags & BB_GETOPT_ERROR) {
                bb_show_usage();
        }
 

diff --git a/debianutils/start_stop_daemon.c b/debianutils/start_stop_daemon.c
index e15944c597beaa98840545a56c290696c5260c41..1eaf0d78c03d82f2433e673fa29e99fdaa81c601 100644 (file)
--- a/debianutils/start_stop_daemon.c
+++ b/debianutils/start_stop_daemon.c
@@ -238,7 +238,7 @@ start_stop_daemon_main(int argc, char **argv)
                        &startas, &cmdname, &signame, &userspec, &execname, &pidfile);
 
        /* Check one and only one context option was given */
-       if ((opt & 0x80000000UL) || (opt & (SSD_CTX_STOP | SSD_CTX_START)) == 0) {
+       if ((opt & BB_GETOPT_ERROR) || (opt & (SSD_CTX_STOP | SSD_CTX_START)) == 0) {
                bb_show_usage();
        }
 

diff --git a/include/libbb.h b/include/libbb.h
index ac60dd6c2061b4aaf9a023d7eccc7114405d0197..1aa7b41bd9358bcca54bd8f541f49c11b932773e 100644 (file)
--- a/include/libbb.h
+++ b/include/libbb.h
@@ -151,6 +151,7 @@ extern FILE *bb_xfopen(const char *path, const char *mode);
 extern int   bb_fclose_nonstdin(FILE *f);
 extern void  bb_fflush_stdout_and_exit(int retval) __attribute__ ((noreturn));
 
+#define BB_GETOPT_ERROR 0x80000000UL
 extern const char *bb_opt_complementaly;
 extern const struct option *bb_applet_long_options;
 extern unsigned long bb_getopt_ulflags(int argc, char **argv, const char *applet_opts, ...);

diff --git a/libbb/getopt_ulflags.c b/libbb/getopt_ulflags.c
index b6a35261d23951f46799b1c2bc65d0e60fbcdac7..6197e8d9f18fc3e421bae25dc9729bddbaab0c5a 100644 (file)
--- a/libbb/getopt_ulflags.c
+++ b/libbb/getopt_ulflags.c
@@ -155,10 +155,10 @@ Special characters:
            if specified together.  In this case you must set
            bb_opt_complementaly = "b~cf:c~bf:f~bc".  If two of the
            mutually exclusive options are found, bb_getopt_ulflags's
-           return value will have the error flag set (0x80000000UL) so
+           return value will have the error flag set (BB_GETOPT_ERROR) so
            that we can check for it:
 
-           if (flags & 0x80000000UL)
+           if (flags & BB_GETOPT_ERROR)
                    bb_show_usage();
 
    "*"     A star after a char in bb_opt_complementaly means that the
@@ -280,7 +280,7 @@ bb_getopt_ulflags (int argc, char **argv, const char *applet_opts, ...)
                                bb_show_usage ();
                }
                if(flags & on_off->incongruously)
-                       flags |= 0x80000000UL;
+                       flags |= BB_GETOPT_ERROR;
                flags &= ~on_off->switch_off;
                flags |= on_off->switch_on;
                if(on_off->list_flg) {

diff --git a/util-linux/hwclock.c b/util-linux/hwclock.c
index 2f626616772fcdb7f4ee8a572cd109e374947761..3c26839537caba155dceff4a0b17fcc9f97f68ab 100644 (file)
--- a/util-linux/hwclock.c
+++ b/util-linux/hwclock.c
@@ -208,7 +208,7 @@ static const struct option hwclock_long_options[] = {
        bb_opt_complementaly = "r~ws:w~rs:s~wr:l~u:u~l";
        opt = bb_getopt_ulflags(argc, argv, "lursw");
        /* Check only one mode was given */
-       if(opt & 0x80000000UL) {
+       if(opt & BB_GETOPT_ERROR) {
                bb_show_usage();
        }