diff options
author | Denys Vlasenko <vda.linux@googlemail.com> | 2015-03-12 17:48:34 +0100 |
---|---|---|
committer | Denys Vlasenko <vda.linux@googlemail.com> | 2015-03-12 17:48:34 +0100 |
commit | 8dff01d06a7ebd7330e3a1dd1ba47b3c74ee7dfb (patch) | |
tree | 41c9dfbd45676c9e8737a7d8aa4361bbdfb40868 /miscutils | |
parent | 936c8809caea5705e26e5d7e06ea3895c28fffd8 (diff) | |
download | busybox-w32-8dff01d06a7ebd7330e3a1dd1ba47b3c74ee7dfb.tar.gz busybox-w32-8dff01d06a7ebd7330e3a1dd1ba47b3c74ee7dfb.tar.bz2 busybox-w32-8dff01d06a7ebd7330e3a1dd1ba47b3c74ee7dfb.zip |
libbb: introduce and use is_prefixed_with()
function old new delta
is_prefixed_with - 18 +18
complete_username 78 77 -1
man_main 737 735 -2
fsck_device 429 427 -2
unpack_ar_archive 80 76 -4
strip_unsafe_prefix 105 101 -4
singlemount 1054 1050 -4
rtc_adjtime_is_utc 90 86 -4
resolve_mount_spec 88 84 -4
parse_one_line 1029 1025 -4
parse_conf 1460 1456 -4
may_wakeup 83 79 -4
loadkmap_main 219 215 -4
get_irqs_from_stat 103 99 -4
get_header_cpio 913 909 -4
findfs_main 79 75 -4
fbsplash_main 1230 1226 -4
load_crontab 776 771 -5
expand_vars_to_list 1151 1146 -5
date_main 881 876 -5
skip_dev_pfx 30 24 -6
make_device 2199 2193 -6
complete_cmd_dir_file 773 767 -6
run_applet_and_exit 715 708 -7
uudecode_main 321 313 -8
pwdx_main 197 189 -8
execute 568 560 -8
i2cdetect_main 1186 1176 -10
procps_scan 1242 1230 -12
procps_read_smaps 1017 1005 -12
process_module 746 734 -12
patch_main 1903 1891 -12
nfsmount 3572 3560 -12
stack_machine 126 112 -14
process_timer_stats 449 435 -14
match_fstype 111 97 -14
do_ipaddr 1344 1330 -14
open_list_and_close 359 343 -16
get_header_tar 1795 1779 -16
prepend_new_eth_table 340 323 -17
fsck_main 1811 1794 -17
find_iface_state 56 38 -18
dnsd_main 1321 1303 -18
base_device 179 158 -21
find_keyword 104 82 -22
handle_incoming_and_exit 2785 2762 -23
parse_and_put_prompt 774 746 -28
modinfo 347 317 -30
find_action 204 171 -33
update_passwd 1470 1436 -34
------------------------------------------------------------------------------
(add/remove: 1/0 grow/shrink: 0/49 up/down: 18/-540) Total: -522 bytes
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
Diffstat (limited to 'miscutils')
-rw-r--r-- | miscutils/crond.c | 4 | ||||
-rw-r--r-- | miscutils/dc.c | 6 | ||||
-rw-r--r-- | miscutils/devfsd.c | 10 | ||||
-rw-r--r-- | miscutils/fbsplash.c | 2 | ||||
-rw-r--r-- | miscutils/i2c_tools.c | 4 | ||||
-rw-r--r-- | miscutils/last.c | 6 | ||||
-rw-r--r-- | miscutils/man.c | 4 |
7 files changed, 17 insertions, 19 deletions
diff --git a/miscutils/crond.c b/miscutils/crond.c index 3659b9a6f..eb327f855 100644 --- a/miscutils/crond.c +++ b/miscutils/crond.c | |||
@@ -438,14 +438,14 @@ static void load_crontab(const char *fileName) | |||
438 | log5("user:%s entry:%s", fileName, parser->data); | 438 | log5("user:%s entry:%s", fileName, parser->data); |
439 | 439 | ||
440 | /* check if line is setting MAILTO= */ | 440 | /* check if line is setting MAILTO= */ |
441 | if (0 == strncmp(tokens[0], "MAILTO=", 7)) { | 441 | if (is_prefixed_with(tokens[0], "MAILTO=")) { |
442 | #if ENABLE_FEATURE_CROND_CALL_SENDMAIL | 442 | #if ENABLE_FEATURE_CROND_CALL_SENDMAIL |
443 | free(mailTo); | 443 | free(mailTo); |
444 | mailTo = (tokens[0][7]) ? xstrdup(&tokens[0][7]) : NULL; | 444 | mailTo = (tokens[0][7]) ? xstrdup(&tokens[0][7]) : NULL; |
445 | #endif /* otherwise just ignore such lines */ | 445 | #endif /* otherwise just ignore such lines */ |
446 | continue; | 446 | continue; |
447 | } | 447 | } |
448 | if (0 == strncmp(tokens[0], "SHELL=", 6)) { | 448 | if (is_prefixed_with(tokens[0], "SHELL=")) { |
449 | free(shell); | 449 | free(shell); |
450 | shell = xstrdup(&tokens[0][6]); | 450 | shell = xstrdup(&tokens[0][6]); |
451 | continue; | 451 | continue; |
diff --git a/miscutils/dc.c b/miscutils/dc.c index a7bd360d2..f94d6fa6b 100644 --- a/miscutils/dc.c +++ b/miscutils/dc.c | |||
@@ -244,9 +244,9 @@ static void stack_machine(const char *argument) | |||
244 | 244 | ||
245 | o = operators; | 245 | o = operators; |
246 | do { | 246 | do { |
247 | const size_t name_len = strlen(o->name); | 247 | char *after_name = is_prefixed_with(argument, o->name); |
248 | if (strncmp(o->name, argument, name_len) == 0) { | 248 | if (after_name) { |
249 | argument += name_len; | 249 | argument = after_name; |
250 | o->function(); | 250 | o->function(); |
251 | goto next; | 251 | goto next; |
252 | } | 252 | } |
diff --git a/miscutils/devfsd.c b/miscutils/devfsd.c index 96ffe0738..5a6aec6bd 100644 --- a/miscutils/devfsd.c +++ b/miscutils/devfsd.c | |||
@@ -1405,7 +1405,6 @@ const char *get_old_name(const char *devname, unsigned int namelen, | |||
1405 | int indexx; | 1405 | int indexx; |
1406 | const char *pty1; | 1406 | const char *pty1; |
1407 | const char *pty2; | 1407 | const char *pty2; |
1408 | size_t len; | ||
1409 | /* 1 to 5 "scsi/" , 6 to 9 "ide/host", 10 sbp/, 11 vcc/, 12 pty/ */ | 1408 | /* 1 to 5 "scsi/" , 6 to 9 "ide/host", 10 sbp/, 11 vcc/, 12 pty/ */ |
1410 | static const char *const fmt[] = { | 1409 | static const char *const fmt[] = { |
1411 | NULL , | 1410 | NULL , |
@@ -1425,12 +1424,11 @@ const char *get_old_name(const char *devname, unsigned int namelen, | |||
1425 | }; | 1424 | }; |
1426 | 1425 | ||
1427 | for (trans = translate_table; trans->match != NULL; ++trans) { | 1426 | for (trans = translate_table; trans->match != NULL; ++trans) { |
1428 | len = strlen(trans->match); | 1427 | char *after_match = is_prefixed_with(devname, trans->match); |
1429 | 1428 | if (after_match) { | |
1430 | if (strncmp(devname, trans->match, len) == 0) { | ||
1431 | if (trans->format == NULL) | 1429 | if (trans->format == NULL) |
1432 | return devname + len; | 1430 | return after_match; |
1433 | sprintf(buffer, trans->format, devname + len); | 1431 | sprintf(buffer, trans->format, after_match); |
1434 | return buffer; | 1432 | return buffer; |
1435 | } | 1433 | } |
1436 | } | 1434 | } |
diff --git a/miscutils/fbsplash.c b/miscutils/fbsplash.c index 7b695b26f..77033c258 100644 --- a/miscutils/fbsplash.c +++ b/miscutils/fbsplash.c | |||
@@ -516,7 +516,7 @@ int fbsplash_main(int argc UNUSED_PARAM, char **argv) | |||
516 | // handle a case when we have many buffered lines | 516 | // handle a case when we have many buffered lines |
517 | // already in the pipe | 517 | // already in the pipe |
518 | while ((num_buf = xmalloc_fgetline(fp)) != NULL) { | 518 | while ((num_buf = xmalloc_fgetline(fp)) != NULL) { |
519 | if (strncmp(num_buf, "exit", 4) == 0) { | 519 | if (is_prefixed_with(num_buf, "exit")) { |
520 | DEBUG_MESSAGE("exit"); | 520 | DEBUG_MESSAGE("exit"); |
521 | break; | 521 | break; |
522 | } | 522 | } |
diff --git a/miscutils/i2c_tools.c b/miscutils/i2c_tools.c index 90d1e1e14..03bb03974 100644 --- a/miscutils/i2c_tools.c +++ b/miscutils/i2c_tools.c | |||
@@ -1198,7 +1198,7 @@ static void NORETURN list_i2c_busses_and_exit(void) | |||
1198 | if (subde->d_name[0] == '.') | 1198 | if (subde->d_name[0] == '.') |
1199 | continue; | 1199 | continue; |
1200 | 1200 | ||
1201 | if (strncmp(subde->d_name, "i2c-", 4) == 0) { | 1201 | if (is_prefixed_with(subde->d_name, "i2c-")) { |
1202 | snprintf(path, NAME_MAX, | 1202 | snprintf(path, NAME_MAX, |
1203 | "%s/%s/device/%s/name", | 1203 | "%s/%s/device/%s/name", |
1204 | i2cdev_path, de->d_name, | 1204 | i2cdev_path, de->d_name, |
@@ -1229,7 +1229,7 @@ found: | |||
1229 | if (rv != 1) | 1229 | if (rv != 1) |
1230 | continue; | 1230 | continue; |
1231 | 1231 | ||
1232 | if (strncmp(name, "ISA", 3) == 0) | 1232 | if (is_prefixed_with(name, "ISA")) |
1233 | adt = ADT_ISA; | 1233 | adt = ADT_ISA; |
1234 | else | 1234 | else |
1235 | adt = i2cdetect_get_funcs(bus); | 1235 | adt = i2cdetect_get_funcs(bus); |
diff --git a/miscutils/last.c b/miscutils/last.c index 24f6e1c78..a144c7e47 100644 --- a/miscutils/last.c +++ b/miscutils/last.c | |||
@@ -87,11 +87,11 @@ int last_main(int argc UNUSED_PARAM, char **argv UNUSED_PARAM) | |||
87 | if (++n > 0) | 87 | if (++n > 0) |
88 | ut.ut_type = n != 3 ? n : SHUTDOWN_TIME; | 88 | ut.ut_type = n != 3 ? n : SHUTDOWN_TIME; |
89 | #else | 89 | #else |
90 | if (strncmp(ut.ut_user, "shutdown", 8) == 0) | 90 | if (is_prefixed_with(ut.ut_user, "shutdown")) |
91 | ut.ut_type = SHUTDOWN_TIME; | 91 | ut.ut_type = SHUTDOWN_TIME; |
92 | else if (strncmp(ut.ut_user, "reboot", 6) == 0) | 92 | else if (is_prefixed_with(ut.ut_user, "reboot")) |
93 | ut.ut_type = BOOT_TIME; | 93 | ut.ut_type = BOOT_TIME; |
94 | else if (strncmp(ut.ut_user, "runlevel", 8) == 0) | 94 | else if (is_prefixed_with(ut.ut_user, "runlevel")) |
95 | ut.ut_type = RUN_LVL; | 95 | ut.ut_type = RUN_LVL; |
96 | #endif | 96 | #endif |
97 | } else { | 97 | } else { |
diff --git a/miscutils/man.c b/miscutils/man.c index c39870e67..58ed81955 100644 --- a/miscutils/man.c +++ b/miscutils/man.c | |||
@@ -66,7 +66,7 @@ static int run_pipe(const char *pager, char *man_filename, int man, int level) | |||
66 | goto ordinary_manpage; | 66 | goto ordinary_manpage; |
67 | 67 | ||
68 | line = xmalloc_open_zipped_read_close(man_filename, NULL); | 68 | line = xmalloc_open_zipped_read_close(man_filename, NULL); |
69 | if (!line || strncmp(line, ".so ", 4) != 0) { | 69 | if (!line || !is_prefixed_with(line, ".so ")) { |
70 | free(line); | 70 | free(line); |
71 | goto ordinary_manpage; | 71 | goto ordinary_manpage; |
72 | } | 72 | } |
@@ -228,7 +228,7 @@ int man_main(int argc UNUSED_PARAM, char **argv) | |||
228 | if (!token[1]) | 228 | if (!token[1]) |
229 | continue; | 229 | continue; |
230 | if (strcmp("DEFINE", token[0]) == 0) { | 230 | if (strcmp("DEFINE", token[0]) == 0) { |
231 | if (strncmp("pager", token[1], 5) == 0) { | 231 | if (is_prefixed_with("pager", token[1])) { |
232 | pager = xstrdup(skip_whitespace(token[1]) + 5); | 232 | pager = xstrdup(skip_whitespace(token[1]) + 5); |
233 | } | 233 | } |
234 | } else | 234 | } else |