diff options
author | Bernhard Reutner-Fischer <rep.dot.nop@gmail.com> | 2006-04-03 09:46:29 +0000 |
---|---|---|
committer | Bernhard Reutner-Fischer <rep.dot.nop@gmail.com> | 2006-04-03 09:46:29 +0000 |
commit | 6153629097cdbf463ed50181df5113438662a425 (patch) | |
tree | 0bd0f68cca4ec61ea4e56cfadab5387672c38f02 | |
parent | cfb53dfd5ec455bc0299654d4bb06b30dd5d1373 (diff) | |
download | busybox-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.c | 21 |
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 */ |