diff options
author | Denis Vlasenko <vda.linux@googlemail.com> | 2006-10-03 21:00:06 +0000 |
---|---|---|
committer | Denis Vlasenko <vda.linux@googlemail.com> | 2006-10-03 21:00:06 +0000 |
commit | 67b23e6043d8e2b30b0bf3bc105b8583c2a26db5 (patch) | |
tree | edb58560b444979051b42ab7f0c0c718f7459754 /findutils | |
parent | 40920825d59874cf285390434486e88c8498d2d8 (diff) | |
download | busybox-w32-67b23e6043d8e2b30b0bf3bc105b8583c2a26db5.tar.gz busybox-w32-67b23e6043d8e2b30b0bf3bc105b8583c2a26db5.tar.bz2 busybox-w32-67b23e6043d8e2b30b0bf3bc105b8583c2a26db5.zip |
getopt_ulflags -> getopt32.
It is impossible to formulate sane ABI based on
size of ulong because it can be 32-bit or 64-bit.
Basically it means that you cannot portably use
more that 32 option chars in one call anyway...
Make it explicit.
Diffstat (limited to 'findutils')
-rw-r--r-- | findutils/grep.c | 10 | ||||
-rw-r--r-- | findutils/xargs.c | 4 |
2 files changed, 7 insertions, 7 deletions
diff --git a/findutils/grep.c b/findutils/grep.c index 5659a7d2b..35e6aff2d 100644 --- a/findutils/grep.c +++ b/findutils/grep.c | |||
@@ -24,7 +24,7 @@ | |||
24 | 24 | ||
25 | 25 | ||
26 | /* options */ | 26 | /* options */ |
27 | static unsigned long opt; | 27 | static unsigned opt; |
28 | #define GREP_OPTS "lnqvscFiHhe:f:Lo" | 28 | #define GREP_OPTS "lnqvscFiHhe:f:Lo" |
29 | #define GREP_OPT_l (1<<0) | 29 | #define GREP_OPT_l (1<<0) |
30 | #define PRINT_FILES_WITH_MATCHES (opt & GREP_OPT_l) | 30 | #define PRINT_FILES_WITH_MATCHES (opt & GREP_OPT_l) |
@@ -302,8 +302,8 @@ int grep_main(int argc, char **argv) | |||
302 | char *slines_before; | 302 | char *slines_before; |
303 | char *Copt; | 303 | char *Copt; |
304 | 304 | ||
305 | bb_opt_complementally = "H-h:e::f::C-AB"; | 305 | opt_complementary = "H-h:e::f::C-AB"; |
306 | opt = bb_getopt_ulflags(argc, argv, | 306 | opt = getopt32(argc, argv, |
307 | GREP_OPTS GREP_OPT_CONTEXT OPT_EGREP, | 307 | GREP_OPTS GREP_OPT_CONTEXT OPT_EGREP, |
308 | &pattern_head, &fopt, | 308 | &pattern_head, &fopt, |
309 | &slines_after, &slines_before, &Copt); | 309 | &slines_after, &slines_before, &Copt); |
@@ -336,8 +336,8 @@ int grep_main(int argc, char **argv) | |||
336 | before_buf = (char **)xzalloc(lines_before * sizeof(char *)); | 336 | before_buf = (char **)xzalloc(lines_before * sizeof(char *)); |
337 | #else | 337 | #else |
338 | /* with auto sanity checks */ | 338 | /* with auto sanity checks */ |
339 | bb_opt_complementally = "H-h:e::f::c-n:q-n:l-n"; | 339 | opt_complementary = "H-h:e::f::c-n:q-n:l-n"; |
340 | opt = bb_getopt_ulflags(argc, argv, GREP_OPTS OPT_EGREP, | 340 | opt = getopt32(argc, argv, GREP_OPTS OPT_EGREP, |
341 | &pattern_head, &fopt); | 341 | &pattern_head, &fopt); |
342 | #endif | 342 | #endif |
343 | invert_search = (opt & GREP_OPT_v) != 0; /* 0 | 1 */ | 343 | invert_search = (opt & GREP_OPT_v) != 0; /* 0 | 1 */ |
diff --git a/findutils/xargs.c b/findutils/xargs.c index b6a154f15..278382373 100644 --- a/findutils/xargs.c +++ b/findutils/xargs.c | |||
@@ -399,7 +399,7 @@ int xargs_main(int argc, char **argv) | |||
399 | size_t n_chars = 0; | 399 | size_t n_chars = 0; |
400 | long orig_arg_max; | 400 | long orig_arg_max; |
401 | const char *eof_str = "_"; | 401 | const char *eof_str = "_"; |
402 | unsigned long opt; | 402 | unsigned opt; |
403 | size_t n_max_chars; | 403 | size_t n_max_chars; |
404 | #if ENABLE_FEATURE_XARGS_SUPPORT_ZERO_TERM | 404 | #if ENABLE_FEATURE_XARGS_SUPPORT_ZERO_TERM |
405 | xlist_t* (*read_args)(xlist_t*, const char*, size_t, char*) = process_stdin; | 405 | xlist_t* (*read_args)(xlist_t*, const char*, size_t, char*) = process_stdin; |
@@ -407,7 +407,7 @@ int xargs_main(int argc, char **argv) | |||
407 | #define read_args process_stdin | 407 | #define read_args process_stdin |
408 | #endif | 408 | #endif |
409 | 409 | ||
410 | opt = bb_getopt_ulflags(argc, argv, OPTION_STR, &max_args, &max_chars, &eof_str); | 410 | opt = getopt32(argc, argv, OPTION_STR, &max_args, &max_chars, &eof_str); |
411 | 411 | ||
412 | if (opt & OPT_ZEROTERM) | 412 | if (opt & OPT_ZEROTERM) |
413 | USE_FEATURE_XARGS_SUPPORT_ZERO_TERM(read_args = process0_stdin); | 413 | USE_FEATURE_XARGS_SUPPORT_ZERO_TERM(read_args = process0_stdin); |