diff options
author | Junio C Hamano <junkio@cox.net> | 2006-03-05 02:47:29 -0800 |
---|---|---|
committer | Junio C Hamano <junkio@cox.net> | 2006-03-05 02:47:29 -0800 |
commit | 9201c707426b3dc0c894775416f576c25c008d46 (patch) | |
tree | 235727178999b09a9d26ff4c384199558761fe15 /exec_cmd.c | |
parent | 4a5d6939509f9aeef600ea17643caef4c898b12f (diff) | |
download | git-9201c707426b3dc0c894775416f576c25c008d46.tar.gz git-9201c707426b3dc0c894775416f576c25c008d46.tar.xz |
Const tightening.
Mark Wooding noticed there was a type mismatch warning in git.c; this
patch does things slightly differently (mostly tightening const) and
was what I was holding onto, waiting for the setup-revisions change
to be merged into the master branch.
Signed-off-by: Junio C Hamano <junkio@cox.net>
Diffstat (limited to 'exec_cmd.c')
-rw-r--r-- | exec_cmd.c | 13 |
1 files changed, 7 insertions, 6 deletions
diff --git a/exec_cmd.c b/exec_cmd.c index b5e59a9ae..96cc2123b 100644 --- a/exec_cmd.c +++ b/exec_cmd.c @@ -29,10 +29,9 @@ const char *git_exec_path(void) } -int execv_git_cmd(char **argv) +int execv_git_cmd(const char **argv) { char git_command[PATH_MAX + 1]; - char *tmp; int len, err, i; const char *paths[] = { current_exec_path, getenv("GIT_EXEC_PATH"), @@ -40,6 +39,8 @@ int execv_git_cmd(char **argv) for (i = 0; i < sizeof(paths)/sizeof(paths[0]); ++i) { const char *exec_dir = paths[i]; + const char *tmp; + if (!exec_dir) continue; if (*exec_dir != '/') { @@ -82,7 +83,7 @@ int execv_git_cmd(char **argv) argv[0] = git_command; /* execve() can only ever return if it fails */ - execve(git_command, argv, environ); + execve(git_command, (char **)argv, environ); err = errno; @@ -93,11 +94,11 @@ int execv_git_cmd(char **argv) } -int execl_git_cmd(char *cmd,...) +int execl_git_cmd(const char *cmd,...) { int argc; - char *argv[MAX_ARGS + 1]; - char *arg; + const char *argv[MAX_ARGS + 1]; + const char *arg; va_list param; va_start(param, cmd); |