diff options
author | Denys Vlasenko <vda.linux@googlemail.com> | 2009-09-23 17:17:53 +0200 |
---|---|---|
committer | Denys Vlasenko <vda.linux@googlemail.com> | 2009-09-23 17:17:53 +0200 |
commit | 1f27ab0d4bb65425496ff4ed0fbbd0f5bb32786f (patch) | |
tree | a7181ba3c498570257040c8663f125938ecad0a6 /networking/libiproute | |
parent | 8d338173a4668740b1ab4a40d1d26cd25402e406 (diff) | |
download | busybox-w32-1f27ab0d4bb65425496ff4ed0fbbd0f5bb32786f.tar.gz busybox-w32-1f27ab0d4bb65425496ff4ed0fbbd0f5bb32786f.tar.bz2 busybox-w32-1f27ab0d4bb65425496ff4ed0fbbd0f5bb32786f.zip |
*: optimize code size in strtoul calls
function old new delta
bb_parse_mode 433 431 -2
rtnl_rtntype_a2n 202 198 -4
ParseField 511 498 -13
bb_init_module_24 4730 4675 -55
------------------------------------------------------------------------------
(add/remove: 0/0 grow/shrink: 0/4 up/down: 0/-74) Total: -74 bytes
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
Diffstat (limited to 'networking/libiproute')
-rw-r--r-- | networking/libiproute/rtm_map.c | 2 | ||||
-rw-r--r-- | networking/libiproute/utils.c | 3 |
2 files changed, 4 insertions, 1 deletions
diff --git a/networking/libiproute/rtm_map.c b/networking/libiproute/rtm_map.c index ca2f4436a..6fe5c4b75 100644 --- a/networking/libiproute/rtm_map.c +++ b/networking/libiproute/rtm_map.c | |||
@@ -88,7 +88,7 @@ int rtnl_rtntype_a2n(int *id, char *arg) | |||
88 | res = RTN_THROW; | 88 | res = RTN_THROW; |
89 | else { | 89 | else { |
90 | res = strtoul(arg, &end, 0); | 90 | res = strtoul(arg, &end, 0); |
91 | if (!end || end == arg || *end || res > 255) | 91 | if (end == arg || *end || res > 255) |
92 | return -1; | 92 | return -1; |
93 | } | 93 | } |
94 | *id = res; | 94 | *id = res; |
diff --git a/networking/libiproute/utils.c b/networking/libiproute/utils.c index c84d018eb..5f0971751 100644 --- a/networking/libiproute/utils.c +++ b/networking/libiproute/utils.c | |||
@@ -22,6 +22,7 @@ unsigned get_unsigned(char *arg, const char *errmsg) | |||
22 | 22 | ||
23 | if (*arg) { | 23 | if (*arg) { |
24 | res = strtoul(arg, &ptr, 0); | 24 | res = strtoul(arg, &ptr, 0); |
25 | //FIXME: "" will be accepted too, is it correct?! | ||
25 | if (!*ptr && res <= UINT_MAX) { | 26 | if (!*ptr && res <= UINT_MAX) { |
26 | return res; | 27 | return res; |
27 | } | 28 | } |
@@ -36,6 +37,7 @@ uint32_t get_u32(char *arg, const char *errmsg) | |||
36 | 37 | ||
37 | if (*arg) { | 38 | if (*arg) { |
38 | res = strtoul(arg, &ptr, 0); | 39 | res = strtoul(arg, &ptr, 0); |
40 | //FIXME: "" will be accepted too, is it correct?! | ||
39 | if (!*ptr && res <= 0xFFFFFFFFUL) { | 41 | if (!*ptr && res <= 0xFFFFFFFFUL) { |
40 | return res; | 42 | return res; |
41 | } | 43 | } |
@@ -50,6 +52,7 @@ uint16_t get_u16(char *arg, const char *errmsg) | |||
50 | 52 | ||
51 | if (*arg) { | 53 | if (*arg) { |
52 | res = strtoul(arg, &ptr, 0); | 54 | res = strtoul(arg, &ptr, 0); |
55 | //FIXME: "" will be accepted too, is it correct?! | ||
53 | if (!*ptr && res <= 0xFFFF) { | 56 | if (!*ptr && res <= 0xFFFF) { |
54 | return res; | 57 | return res; |
55 | } | 58 | } |