emo/busybox
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

*: make 2 more "int argc"'s unused; more saved if !DESKTOP

Browse Source 
function                                             old     new   delta
sync_main                                             18      20      +2
tty_main                                              90      87      -3
iprule_list                                           83      79      -4

Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
This commit is contained in:
Denys Vlasenko 2010-01-04 13:16:08 +01:00
parent a4899efd03
commit a355da0775
7 changed files with 20 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
console-tools/dumpkmap.c
View File

@ -28,9 +28,9 @@ int dumpkmap_main(int argc UNUSED_PARAM, char **argv UNUSED_PARAM)
{ {
struct kbentry ke; struct kbentry ke;
int i, j, fd; int i, j, fd;
RESERVE_CONFIG_BUFFER(flags,MAX_NR_KEYMAPS); RESERVE_CONFIG_BUFFER(flags, MAX_NR_KEYMAPS);
/* bb_warn_ignoring_args(argc>=2);*/ /* bb_warn_ignoring_args(argv[1]);*/
fd = get_console_fd_or_die(); fd = get_console_fd_or_die();

4
console-tools/loadkmap.c
View File

@ -30,9 +30,9 @@ int loadkmap_main(int argc UNUSED_PARAM, char **argv UNUSED_PARAM)
int i, j, fd; int i, j, fd;
uint16_t ibuff[NR_KEYS]; uint16_t ibuff[NR_KEYS];
/* const char *tty_name = CURRENT_TTY; */ /* const char *tty_name = CURRENT_TTY; */
RESERVE_CONFIG_BUFFER(flags,MAX_NR_KEYMAPS); RESERVE_CONFIG_BUFFER(flags, MAX_NR_KEYMAPS);
/* bb_warn_ignoring_args(argc >= 2); */ /* bb_warn_ignoring_args(argv[1]); */
fd = get_console_fd_or_die(); fd = get_console_fd_or_die();
/* or maybe: /* or maybe:
opt = getopt32(argv, "C:", &tty_name); opt = getopt32(argv, "C:", &tty_name);

4
coreutils/sync.c
View File

@ -14,10 +14,10 @@
/* This is a NOFORK applet. Be very careful! */ /* This is a NOFORK applet. Be very careful! */
int sync_main(int argc, char **argv) MAIN_EXTERNALLY_VISIBLE; int sync_main(int argc, char **argv) MAIN_EXTERNALLY_VISIBLE;
int sync_main(int argc, char **argv UNUSED_PARAM) int sync_main(int argc UNUSED_PARAM, char **argv)
{ {
/* coreutils-6.9 compat */ /* coreutils-6.9 compat */
bb_warn_ignoring_args(argc - 1); bb_warn_ignoring_args(argv[1]);
sync(); sync();

8
coreutils/tty.c
View File

@ -13,7 +13,7 @@
#include "libbb.h" #include "libbb.h"
int tty_main(int argc, char **argv) MAIN_EXTERNALLY_VISIBLE; int tty_main(int argc, char **argv) MAIN_EXTERNALLY_VISIBLE;
int tty_main(int argc, char **argv IF_NOT_INCLUDE_SUSv2(UNUSED_PARAM)) int tty_main(int argc UNUSED_PARAM, char **argv)
{ {
const char *s; const char *s;
IF_INCLUDE_SUSv2(int silent;) /* Note: No longer relevant in SUSv3. */ IF_INCLUDE_SUSv2(int silent;) /* Note: No longer relevant in SUSv3. */
@ -22,11 +22,11 @@ int tty_main(int argc, char **argv IF_NOT_INCLUDE_SUSv2(UNUSED_PARAM))
xfunc_error_retval = 2; /* SUSv3 requires > 1 for error. */ xfunc_error_retval = 2; /* SUSv3 requires > 1 for error. */
IF_INCLUDE_SUSv2(silent = getopt32(argv, "s");) IF_INCLUDE_SUSv2(silent = getopt32(argv, "s");)
IF_INCLUDE_SUSv2(argc -= optind;) IF_INCLUDE_SUSv2(argv += optind;)
IF_NOT_INCLUDE_SUSv2(argc -= 1;) IF_NOT_INCLUDE_SUSv2(argv += 1;)
/* gnu tty outputs a warning that it is ignoring all args. */ /* gnu tty outputs a warning that it is ignoring all args. */
bb_warn_ignoring_args(argc); bb_warn_ignoring_args(argv[0]);
retval = EXIT_SUCCESS; retval = EXIT_SUCCESS;

6
include/libbb.h
View File

@ -1022,7 +1022,11 @@ extern struct mntent *find_mount_point(const char *name, int subdir_too) FAST_FU
extern void erase_mtab(const char * name) FAST_FUNC; extern void erase_mtab(const char * name) FAST_FUNC;
extern unsigned int tty_baud_to_value(speed_t speed) FAST_FUNC; extern unsigned int tty_baud_to_value(speed_t speed) FAST_FUNC;
extern speed_t tty_value_to_baud(unsigned int value) FAST_FUNC; extern speed_t tty_value_to_baud(unsigned int value) FAST_FUNC;
extern void bb_warn_ignoring_args(int n) FAST_FUNC; #if ENABLE_DESKTOP
extern void bb_warn_ignoring_args(char *arg) FAST_FUNC;
#else
# define bb_warn_ignoring_args(arg) ((void)0)
#endif
extern int get_linux_version_code(void) FAST_FUNC; extern int get_linux_version_code(void) FAST_FUNC;

7
libbb/warn_ignoring_args.c
View File

@ -6,12 +6,13 @@
* *
* Licensed under GPLv2 or later, see file LICENSE in this tarball for details. * Licensed under GPLv2 or later, see file LICENSE in this tarball for details.
*/ */
#include "libbb.h" #include "libbb.h"
void FAST_FUNC bb_warn_ignoring_args(int n) #if ENABLE_DESKTOP
void FAST_FUNC bb_warn_ignoring_args(char *arg)
{ {
if (n) { if (arg) {
bb_error_msg("ignoring all arguments"); bb_error_msg("ignoring all arguments");
} }
} }
#endif

2
networking/libiproute/iprule.c
View File

@ -166,7 +166,7 @@ static int iprule_list(char **argv)
if (*argv) { if (*argv) {
//bb_error_msg("\"rule show\" needs no arguments"); //bb_error_msg("\"rule show\" needs no arguments");
bb_warn_ignoring_args(1); bb_warn_ignoring_args(*argv);
return -1; return -1;
} }