diff options
author | Denys Vlasenko <vda.linux@googlemail.com> | 2016-07-06 21:58:02 +0200 |
---|---|---|
committer | Denys Vlasenko <vda.linux@googlemail.com> | 2016-07-06 21:58:02 +0200 |
commit | 237bedd499c58034a1355484d6d4d906f0180308 (patch) | |
tree | 42a789f8867ba3bc6ef0cea30fb18f5553d447a0 /editors | |
parent | 0844b5afe2cd60d46b7f2ad0fac8c2818d0780b3 (diff) | |
download | busybox-w32-237bedd499c58034a1355484d6d4d906f0180308.tar.gz busybox-w32-237bedd499c58034a1355484d6d4d906f0180308.tar.bz2 busybox-w32-237bedd499c58034a1355484d6d4d906f0180308.zip |
getopt32: add new syntax of 'o:+' and 'o:*' for -o NUM and -o LIST
In many cases, this aqllows to drop use of opt_complementary.
Approximately -400 bytes:
function old new delta
getopt32 1423 1502 +79
opt_string 17 18 +1
OPT_STR 24 25 +1
uniq_main 416 406 -10
timeout_main 279 269 -10
sulogin_main 270 260 -10
readprofile_main 1825 1815 -10
ps_main 543 533 -10
pidof_main 245 235 -10
pgrep_main 611 601 -10
od_main 2600 2590 -10
mkfs_minix_main 2684 2674 -10
mkfs_ext2_main 2603 2593 -10
microcom_main 712 702 -10
makemime_main 315 305 -10
ionice_main 282 272 -10
inetd_main 2074 2064 -10
ifplugd_main 1144 1134 -10
halt_main 353 343 -10
getopt_main 636 626 -10
fdisk_main 2854 2844 -10
env_main 206 196 -10
dmesg_main 319 309 -10
conspy_main 1214 1204 -10
awk_main 981 971 -10
------------------------------------------------------------------------------
(add/remove: 0/0 grow/shrink: 3/22 up/down: 81/-220) Total: -139 bytes
text data bss dec hex filename
919373 906 14060 934339 e41c3 busybox_old
918969 906 14060 933935 e402f busybox_unstripped
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
Diffstat (limited to 'editors')
-rw-r--r-- | editors/awk.c | 8 | ||||
-rw-r--r-- | editors/diff.c | 4 | ||||
-rw-r--r-- | editors/sed.c | 5 |
3 files changed, 6 insertions, 11 deletions
diff --git a/editors/awk.c b/editors/awk.c index 69816464d..d0269b9f4 100644 --- a/editors/awk.c +++ b/editors/awk.c | |||
@@ -72,12 +72,9 @@ | |||
72 | 72 | ||
73 | 73 | ||
74 | #define OPTSTR_AWK \ | 74 | #define OPTSTR_AWK \ |
75 | "F:v:f:" \ | 75 | "F:v:*f:*" \ |
76 | IF_FEATURE_AWK_GNU_EXTENSIONS("e:") \ | 76 | IF_FEATURE_AWK_GNU_EXTENSIONS("e:*") \ |
77 | "W:" | 77 | "W:" |
78 | #define OPTCOMPLSTR_AWK \ | ||
79 | "v::f::" \ | ||
80 | IF_FEATURE_AWK_GNU_EXTENSIONS("e::") | ||
81 | enum { | 78 | enum { |
82 | OPTBIT_F, /* define field separator */ | 79 | OPTBIT_F, /* define field separator */ |
83 | OPTBIT_v, /* define variable */ | 80 | OPTBIT_v, /* define variable */ |
@@ -3209,7 +3206,6 @@ int awk_main(int argc, char **argv) | |||
3209 | *s1 = '='; | 3206 | *s1 = '='; |
3210 | } | 3207 | } |
3211 | } | 3208 | } |
3212 | opt_complementary = OPTCOMPLSTR_AWK; | ||
3213 | opt = getopt32(argv, OPTSTR_AWK, &opt_F, &list_v, &list_f, IF_FEATURE_AWK_GNU_EXTENSIONS(&list_e,) NULL); | 3209 | opt = getopt32(argv, OPTSTR_AWK, &opt_F, &list_v, &list_f, IF_FEATURE_AWK_GNU_EXTENSIONS(&list_e,) NULL); |
3214 | argv += optind; | 3210 | argv += optind; |
3215 | argc -= optind; | 3211 | argc -= optind; |
diff --git a/editors/diff.c b/editors/diff.c index ff269360f..75229ad8c 100644 --- a/editors/diff.c +++ b/editors/diff.c | |||
@@ -982,11 +982,11 @@ int diff_main(int argc UNUSED_PARAM, char **argv) | |||
982 | INIT_G(); | 982 | INIT_G(); |
983 | 983 | ||
984 | /* exactly 2 params; collect multiple -L <label>; -U N */ | 984 | /* exactly 2 params; collect multiple -L <label>; -U N */ |
985 | opt_complementary = "=2:L::U+"; | 985 | opt_complementary = "=2"; |
986 | #if ENABLE_FEATURE_DIFF_LONG_OPTIONS | 986 | #if ENABLE_FEATURE_DIFF_LONG_OPTIONS |
987 | applet_long_options = diff_longopts; | 987 | applet_long_options = diff_longopts; |
988 | #endif | 988 | #endif |
989 | getopt32(argv, "abdiL:NqrsS:tTU:wupBE", | 989 | getopt32(argv, "abdiL:*NqrsS:tTU:+wupBE", |
990 | &L_arg, &s_start, &opt_U_context); | 990 | &L_arg, &s_start, &opt_U_context); |
991 | argv += optind; | 991 | argv += optind; |
992 | while (L_arg) | 992 | while (L_arg) |
diff --git a/editors/sed.c b/editors/sed.c index f37c37d88..c0d79cc7b 100644 --- a/editors/sed.c +++ b/editors/sed.c | |||
@@ -1503,8 +1503,7 @@ int sed_main(int argc UNUSED_PARAM, char **argv) | |||
1503 | /* do normal option parsing */ | 1503 | /* do normal option parsing */ |
1504 | opt_e = opt_f = NULL; | 1504 | opt_e = opt_f = NULL; |
1505 | opt_i = NULL; | 1505 | opt_i = NULL; |
1506 | opt_complementary = "e::f::" /* can occur multiple times */ | 1506 | opt_complementary = "nn"; /* count -n */ |
1507 | "nn"; /* count -n */ | ||
1508 | 1507 | ||
1509 | IF_LONG_OPTS(applet_long_options = sed_longopts); | 1508 | IF_LONG_OPTS(applet_long_options = sed_longopts); |
1510 | 1509 | ||
@@ -1513,7 +1512,7 @@ int sed_main(int argc UNUSED_PARAM, char **argv) | |||
1513 | * GNU sed 4.2.1 mentions it in neither --help | 1512 | * GNU sed 4.2.1 mentions it in neither --help |
1514 | * nor manpage, but does recognize it. | 1513 | * nor manpage, but does recognize it. |
1515 | */ | 1514 | */ |
1516 | opt = getopt32(argv, "i::rEne:f:", &opt_i, &opt_e, &opt_f, | 1515 | opt = getopt32(argv, "i::rEne:*f:*", &opt_i, &opt_e, &opt_f, |
1517 | &G.be_quiet); /* counter for -n */ | 1516 | &G.be_quiet); /* counter for -n */ |
1518 | //argc -= optind; | 1517 | //argc -= optind; |
1519 | argv += optind; | 1518 | argv += optind; |