summaryrefslogtreecommitdiff
path: root/builtin-receive-pack.c
diff options
context:
space:
mode:
authorJohannes Sixt <j6t@kdbg.org>2009-07-04 21:26:39 +0200
committerJunio C Hamano <gitster@pobox.com>2009-07-05 12:16:27 -0700
commit5709e0363a891b72eb9e9756d7fb121d9bf6a7c7 (patch)
tree47c33baed20745fe0f928ed86fd526ac13943622 /builtin-receive-pack.c
parent303e7c48eae7e140a9612ff1f9b5a95ca80b65c4 (diff)
downloadgit-5709e0363a891b72eb9e9756d7fb121d9bf6a7c7.tar.gz
run_command: return exit code as positive value
As a general guideline, functions in git's code return zero to indicate success and negative values to indicate failure. The run_command family of functions followed this guideline. But there are actually two different kinds of failure: - failures of system calls; - non-zero exit code of the program that was run. Usually, a non-zero exit code of the program is a failure and means a failure to the caller. Except that sometimes it does not. For example, the exit code of merge programs (e.g. external merge drivers) conveys information about how the merge failed, and not all exit calls are actually failures. Furthermore, the return value of run_command is sometimes used as exit code by the caller. This change arranges that the exit code of the program is returned as a positive value, which can now be regarded as the "result" of the function. System call failures continue to be reported as negative values. Signed-off-by: Johannes Sixt <j6t@kdbg.org> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin-receive-pack.c')
-rw-r--r--builtin-receive-pack.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/builtin-receive-pack.c b/builtin-receive-pack.c
index 6ec1d056e6..6235903552 100644
--- a/builtin-receive-pack.c
+++ b/builtin-receive-pack.c
@@ -143,8 +143,8 @@ static int run_status(int code, const char *cmd_name)
case -ERR_RUN_COMMAND_WAITPID_NOEXIT:
return error("%s died strangely", cmd_name);
default:
- error("%s exited with error code %d", cmd_name, -code);
- return -code;
+ error("%s exited with error code %d", cmd_name, code);
+ return code;
}
}