From 9c14bb08a434570adc9b2f0f37eac66b92d4c87e Mon Sep 17 00:00:00 2001 From: Jeff King Date: Sat, 9 Apr 2016 13:42:54 -0400 Subject: [PATCH] git_config_set_multivar_in_file: all non-zero returns are errors This function is just a thin wrapper for the "_gently" form of the function. But the gently form is designed to feed builtin/config.c, which passes our return code directly to its exit status, and thus uses positive error values for some cases. We check only negative values, meaning we would fail to die in some cases (e.g., a malformed key). This may or may not be triggerable in practice; we tend to use this non-gentle form only when setting internal variables, which would not have malformed keys. Signed-off-by: Jeff King Signed-off-by: Junio C Hamano --- config.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/config.c b/config.c index 92b0cf447..d349d52f2 100644 --- a/config.c +++ b/config.c @@ -2222,7 +2222,7 @@ void git_config_set_multivar_in_file(const char *config_filename, const char *value_regex, int multi_replace) { if (git_config_set_multivar_in_file_gently(config_filename, key, value, - value_regex, multi_replace) < 0) + value_regex, multi_replace)) die(_("could not set '%s' to '%s'"), key, value); } -- 2.11.0