aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorBernhard Reutner-Fischer <rep.dot.nop@gmail.com>2006-04-03 09:46:29 +0000
committerBernhard Reutner-Fischer <rep.dot.nop@gmail.com>2006-04-03 09:46:29 +0000
commit6153629097cdbf463ed50181df5113438662a425 (patch)
tree0bd0f68cca4ec61ea4e56cfadab5387672c38f02
parentcfb53dfd5ec455bc0299654d4bb06b30dd5d1373 (diff)
downloadbusybox-6153629097cdbf463ed50181df5113438662a425.tar.gz
- set flags in a more reasonable manner. Thanks, vodz.
text data bss dec hex filename 3520 4 96 3620 e24 networking/arping.o.oorig 3488 4 96 3588 e04 networking/arping.o
-rw-r--r--networking/arping.c21
1 files changed, 5 insertions, 16 deletions
diff --git a/networking/arping.c b/networking/arping.c
index 4f1a2e2c5..7618e9630 100644
--- a/networking/arping.c
+++ b/networking/arping.c
@@ -36,8 +36,8 @@ enum cfg_e {
advert = 4,
quiet = 8,
quit_on_reply = 16,
- unicasting = 32,
- broadcast_only = 64
+ broadcast_only = 32,
+ unicasting = 64
};
static int cfg;
@@ -281,22 +281,11 @@ int arping_main(int argc, char **argv)
char *_count, *_timeout, *_device;
opt = bb_getopt_ulflags(argc, argv, "DUAqfbc:w:i:s:",
&_count, &_timeout, &_device);
- if (opt & 1) { /* Dad */
- cfg |= dad;
+ cfg |= opt & 63; /* set respective flags */
+ if (opt & 1) /* Dad also sets quit_on_reply */
cfg |= quit_on_reply;
- }
- if (opt & 2) /* Unsolicited */
- cfg |= unsolicited;
- if (opt & 4) { /* Advert */
- cfg |= advert;
+ if (opt & 4) /* Advert also sets unsolicited */
cfg |= unsolicited;
- }
- if (opt & 8) /* quiet */
- cfg |= quiet;
- if (opt & 16) /* quit on reply */
- cfg |= quit_on_reply;
- if (opt & 32) /* broadcast only */
- cfg |= broadcast_only;
if (opt & 64) /* count */
count = atoi(_count);
if (opt & 128) /* timeout */