*: optimize code size in strtoul calls
authorDenys Vlasenko <vda.linux@googlemail.com>
Wed, 23 Sep 2009 15:17:53 +0000 (17:17 +0200)
committerDenys Vlasenko <vda.linux@googlemail.com>
Wed, 23 Sep 2009 15:17:53 +0000 (17:17 +0200)
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>
libbb/parse_mode.c
libpwdgrp/pwd_grp.c
miscutils/crond.c
modutils/modutils-24.c
networking/libiproute/rtm_map.c
networking/libiproute/utils.c
networking/tc.c
shell/ash.c

index 40105dd3a27d464a01598db88c3b95d50cea0571..6eca00ab11e4eb9df3483984e754d929491191c5 100644 (file)
@@ -40,7 +40,7 @@ int FAST_FUNC bb_parse_mode(const char *s, mode_t *current_mode)
        mode_t new_mode;
        char op;
 
-       if (((unsigned int)(*s - '0')) < 8) {
+       if ((unsigned char)(*s - '0') < 8) {
                unsigned long tmp;
                char *e;
 
index e2077ade0be1ba45c0564705108f12fe7b450501..947f48d432e6e7d6f38e400618055b6a99fea8db 100644 (file)
@@ -816,7 +816,7 @@ static int bb__parsepwent(void *data, char *line)
 
        i = 0;
        do {
-               p = ((char *) ((struct passwd *) data)) + pw_off[i];
+               p = (char *) data + pw_off[i];
 
                if ((i & 6) ^ 2) {      /* i!=2 and i!=3 */
                        *((char **) p) = line;
@@ -873,7 +873,7 @@ static int bb__parsegrent(void *data, char *line)
        end_of_buf = ((struct group *) data)->gr_name; /* Evil hack! */
        i = 0;
        do {
-               p = ((char *) ((struct group *) data)) + gr_off[i];
+               p = (char *) data + gr_off[i];
 
                if (i < 2) {
                        *((char **) p) = line;
@@ -966,15 +966,15 @@ static const unsigned char sp_off[] ALIGN1 = {
        offsetof(struct spwd, sp_flag)          /* 8 - not a char ptr */
 };
 
-static int bb__parsespent(void *data, char * line)
+static int bb__parsespent(void *data, char *line)
 {
        char *endptr;
        char *p;
        int i;
 
        i = 0;
-       do {
-               p = ((char *) ((struct spwd *) data)) + sp_off[i];
+       while (1) {
+               p = (char *) data + sp_off[i];
                if (i < 2) {
                        *((char **) p) = line;
                        line = strchr(line, ':');
@@ -982,10 +982,10 @@ static int bb__parsespent(void *data, char * line)
                                break;
                        }
                } else {
-                       *((long *) p) = (long) strtoul(line, &endptr, 10);
+                       *((long *) p) = strtoul(line, &endptr, 10);
 
                        if (endptr == line) {
-                               *((long *) p) = ((i != 8) ? -1L : ((long)(~0UL)));
+                               *((long *) p) = (i != 8) ? -1L : (long)(~0UL);
                        }
 
                        line = endptr;
@@ -1003,9 +1003,9 @@ static int bb__parsespent(void *data, char * line)
 
                }
 
-               *line++ = 0;
+               *line++ = '\0';
                ++i;
-       } while (1);
+       }
 
        return EINVAL;
 }
index f0b64fe3aaeb9a298c987c6d7385730443626e79..d2104c36f883c9dd2c8f521f5b6c6995c20ee99a 100644 (file)
@@ -320,11 +320,13 @@ static void ParseField(char *user, char *ary, int modvalue, int off,
                        skip = 1;
                        ++ptr;
                } else if (isdigit(*ptr)) {
+                       char *endp;
                        if (n1 < 0) {
-                               n1 = strtol(ptr, &ptr, 10) + off;
+                               n1 = strtol(ptr, &endp, 10) + off;
                        } else {
-                               n2 = strtol(ptr, &ptr, 10) + off;
+                               n2 = strtol(ptr, &endp, 10) + off;
                        }
+                       ptr = endp; /* gcc likes temp var for &endp */
                        skip = 1;
                } else if (names) {
                        int i;
@@ -361,7 +363,9 @@ static void ParseField(char *user, char *ary, int modvalue, int off,
                        n2 = n1;
                }
                if (*ptr == '/') {
-                       skip = strtol(ptr + 1, &ptr, 10);
+                       char *endp;
+                       skip = strtol(ptr + 1, &endp, 10);
+                       ptr = endp; /* gcc likes temp var for &endp */
                }
 
                /*
index 9f91c9979883e719d2bf0a04180a288c634a3fcb..e5ff54d296c6cb96d26e4cf190d26485c485525a 100644 (file)
@@ -2432,11 +2432,11 @@ new_process_module_arguments(struct obj_file *f, const char *options)
                loc = contents + sym->value;
 
                if (*pinfo == 'c') {
-                       if (!isdigit(*(pinfo + 1))) {
+                       if (!isdigit(pinfo[1])) {
                                bb_error_msg_and_die("parameter type 'c' for %s must be followed by"
                                                     " the maximum size", param);
                        }
-                       charssize = strtoul(pinfo + 1, (char **) NULL, 10);
+                       charssize = strtoul(pinfo + 1, NULL, 10);
                }
 
                if (val == NULL) {
@@ -2449,6 +2449,8 @@ new_process_module_arguments(struct obj_file *f, const char *options)
                n = 0;
                p = val;
                while (*p != 0) {
+                       char *endp;
+
                        if (++n > max)
                                bb_error_msg_and_die("too many values for %s (max %d)", param, max);
 
@@ -2472,19 +2474,23 @@ new_process_module_arguments(struct obj_file *f, const char *options)
                                p += len;
                                break;
                        case 'b':
-                               *loc++ = strtoul(p, &p, 0);
+                               *loc++ = strtoul(p, &endp, 0);
+                               p = endp; /* gcc likes temp var for &endp */
                                break;
                        case 'h':
-                               *(short *) loc = strtoul(p, &p, 0);
+                               *(short *) loc = strtoul(p, &endp, 0);
                                loc += tgt_sizeof_short;
+                               p = endp;
                                break;
                        case 'i':
-                               *(int *) loc = strtoul(p, &p, 0);
+                               *(int *) loc = strtoul(p, &endp, 0);
                                loc += tgt_sizeof_int;
+                               p = endp;
                                break;
                        case 'l':
-                               *(long *) loc = strtoul(p, &p, 0);
+                               *(long *) loc = strtoul(p, &endp, 0);
                                loc += tgt_sizeof_long;
+                               p = endp;
                                break;
                        default:
                                bb_error_msg_and_die("unknown parameter type '%c' for %s",
index ca2f4436ad512225967531ec0f27571d14f764d9..6fe5c4b751c31057b13d81c83224a908c4f2af47 100644 (file)
@@ -88,7 +88,7 @@ int rtnl_rtntype_a2n(int *id, char *arg)
                res = RTN_THROW;
        else {
                res = strtoul(arg, &end, 0);
-               if (!end || end == arg || *end || res > 255)
+               if (end == arg || *end || res > 255)
                        return -1;
        }
        *id = res;
index c84d018eb73ae90524c7ebe5814d73d242ccd62a..5f09717515cf62eb86dbd31cdae7d08dfd0fec3b 100644 (file)
@@ -22,6 +22,7 @@ unsigned get_unsigned(char *arg, const char *errmsg)
 
        if (*arg) {
                res = strtoul(arg, &ptr, 0);
+//FIXME: "" will be accepted too, is it correct?!
                if (!*ptr && res <= UINT_MAX) {
                        return res;
                }
@@ -36,6 +37,7 @@ uint32_t get_u32(char *arg, const char *errmsg)
 
        if (*arg) {
                res = strtoul(arg, &ptr, 0);
+//FIXME: "" will be accepted too, is it correct?!
                if (!*ptr && res <= 0xFFFFFFFFUL) {
                        return res;
                }
@@ -50,6 +52,7 @@ uint16_t get_u16(char *arg, const char *errmsg)
 
        if (*arg) {
                res = strtoul(arg, &ptr, 0);
+//FIXME: "" will be accepted too, is it correct?!
                if (!*ptr && res <= 0xFFFF) {
                        return res;
                }
index fc47e9571271c59f1443ebb0858d52a3f3c58281..d9636949c30cf0e4dbc7c5322e6d44964e02d146 100644 (file)
@@ -89,7 +89,7 @@ static int get_qdisc_handle(__u32 *h, const char *str) {
        if (p == str)
                return 1;
        maj <<= 16;
-       if (*p != ':' && *p!=0)
+       if (*p != ':' && *p != '\0')
                return 1;
  ok:
        *h = maj;
@@ -119,7 +119,8 @@ static int get_tc_classid(__u32 *h, const char *str) {
                maj <<= 16;
                str = p + 1;
                min = strtoul(str, &p, 16);
-               if (*p != 0 || min >= (1<<16))
+//FIXME: check for "" too?
+               if (*p != '\0' || min >= (1<<16))
                        return 1;
                maj |= min;
        } else if (*p != 0)
index 68aa675e52ce9a1b71c3b8dcea00a8362db4a403..db28af7d34190e0e2dc6904e21e082a39ce930e2 100644 (file)
@@ -10078,7 +10078,7 @@ change_random(const char *value)
                vrandom.flags &= ~VNOFUNC;
        } else {
                /* set/reset */
-               random_galois_LFSR = random_LCG = strtoul(value, (char **)NULL, 10);
+               random_galois_LFSR = random_LCG = strtoul(value, NULL, 10);
        }
 }
 #endif