summaryrefslogtreecommitdiff
path: root/receive-pack.c
diff options
context:
space:
mode:
authorShawn O. Pearce <spearce@spearce.org>2006-12-30 21:55:15 -0500
committerJunio C Hamano <junkio@cox.net>2006-12-30 22:22:14 -0800
commit9b0b50936ec76ad8e582d18d5bf54bc81c685e9b (patch)
tree1599e84bb2e53adbce3728f62d1c3c38d815a1e9 /receive-pack.c
parentad1a382fbb3ecb1bb017854a470816c815cc46c9 (diff)
downloadgit-9b0b50936ec76ad8e582d18d5bf54bc81c685e9b.tar.gz
Remove unnecessary argc parameter from run_command_v.
The argc parameter is never used by the run_command_v family of functions. Instead they require that the passed argv[] be NULL terminated so they can rely on the operating system's execvp function to correctly pass the arguments to the new process. Making the caller pass the argc is just confusing, as the caller could be mislead into believing that the argc might take precendece over the argv, or that the argv does not need to be NULL terminated. So goodbye argc. Don't come back. Signed-off-by: Shawn O. Pearce <spearce@spearce.org> Signed-off-by: Junio C Hamano <junkio@cox.net>
Diffstat (limited to 'receive-pack.c')
-rw-r--r--receive-pack.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/receive-pack.c b/receive-pack.c
index 59b682c03a..2b0ba638af 100644
--- a/receive-pack.c
+++ b/receive-pack.c
@@ -187,7 +187,7 @@ static void run_update_post_hook(struct command *cmd)
argc++;
}
argv[argc] = NULL;
- run_command_v_opt(argc, argv, RUN_COMMAND_NO_STDIO);
+ run_command_v_opt(argv, RUN_COMMAND_NO_STDIO);
}
/*
@@ -283,7 +283,7 @@ static const char *unpack(void)
unpacker[0] = "unpack-objects";
unpacker[1] = hdr_arg;
unpacker[2] = NULL;
- code = run_command_v_opt(1, unpacker, RUN_GIT_CMD);
+ code = run_command_v_opt(unpacker, RUN_GIT_CMD);
switch (code) {
case 0:
return NULL;