Code

Make argv_copy always allocate its members
authorJonas Fonseca <jonas.fonseca@savoirfairelinux.com>
Wed, 2 Jun 2010 20:29:12 +0000 (16:29 -0400)
committerJonas Fonseca <fonseca@diku.dk>
Thu, 3 Jun 2010 01:34:51 +0000 (21:34 -0400)
tig.c

diff --git a/tig.c b/tig.c
index 3121a3a3847e9ab2deb45a5d27ec6c2990d7af2f..f364930235546fc7c22bcc7e9a596097a94ad67e 100644 (file)
--- a/tig.c
+++ b/tig.c
@@ -682,12 +682,12 @@ argv_free(const char *argv[])
 }
 
 static bool
-argv_copy(const char *dst[], const char *src[], bool allocate)
+argv_copy(const char *dst[], const char *src[])
 {
        int argc;
 
        for (argc = 0; src[argc]; argc++)
-               if (!(dst[argc] = allocate ? strdup(src[argc]) : src[argc]))
+               if (!(dst[argc] = strdup(src[argc])))
                        return FALSE;
        return TRUE;
 }
@@ -1714,7 +1714,7 @@ add_run_request(enum keymap keymap, int key, int argc, const char **argv)
        req->key = key;
        req->argv[0] = NULL;
 
-       if (!argv_copy(req->argv, argv, TRUE))
+       if (!argv_copy(req->argv, argv))
                return REQ_NONE;
 
        return REQ_NONE + ++run_requests;