ash: use mempcpy() in more places
authorDenys Vlasenko <vda.linux@googlemail.com>
Sun, 23 Jul 2017 19:46:02 +0000 (21:46 +0200)
committerDenys Vlasenko <vda.linux@googlemail.com>
Sun, 23 Jul 2017 19:46:34 +0000 (21:46 +0200)
Most changes are taken from dash.

function                                             old     new   delta
single_quote                                         127     129      +2
stack_nputstr                                         28      29      +1
path_advance                                         209     202      -7
rmescapes                                            346     308     -38
------------------------------------------------------------------------------
(add/remove: 0/0 grow/shrink: 2/2 up/down: 3/-45)             Total: -42 bytes

Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
shell/ash.c

index c8b2adf4e7119477cc59ea0b6f70b1dbde1c32bd..fa03f8a4e27c697945fbc93b1b5a3f130ec2808b 100644 (file)
@@ -1658,7 +1658,7 @@ static char *
 stack_nputstr(const char *s, size_t n, char *p)
 {
        p = makestrspace(n, p);
-       p = (char *)memcpy(p, s, n) + n;
+       p = (char *)mempcpy(p, s, n);
        return p;
 }
 
@@ -1761,7 +1761,7 @@ single_quote(const char *s)
                q = p = makestrspace(len + 3, p);
 
                *q++ = '\'';
-               q = (char *)memcpy(q, s, len) + len;
+               q = (char *)mempcpy(q, s, len) + len;
                *q++ = '\'';
                s += len;
 
@@ -1775,7 +1775,7 @@ single_quote(const char *s)
                q = p = makestrspace(len + 3, p);
 
                *q++ = '"';
-               q = (char *)memcpy(q, s - len, len) + len;
+               q = (char *)mempcpy(q, s - len, len);
                *q++ = '"';
 
                STADJUST(q - p, p);
@@ -2453,8 +2453,7 @@ path_advance(const char **path, const char *name)
                growstackblock();
        q = stackblock();
        if (p != start) {
-               memcpy(q, start, p - start);
-               q += p - start;
+               q = mempcpy(q, start, p - start);
                *q++ = '/';
        }
        strcpy(q, name);
@@ -5949,7 +5948,7 @@ rmescapes(char *str, int flag)
                }
                q = r;
                if (len > 0) {
-                       q = (char *)memcpy(q, str, len) + len;
+                       q = (char *)mempcpy(q, str, len);
                }
        }