From: Jeff King Date: Sat, 9 Apr 2016 17:42:54 +0000 (-0400) Subject: git_config_set_multivar_in_file: all non-zero returns are errors X-Git-Tag: v2.8.3~35^2~1 X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=9c14bb08a434570adc9b2f0f37eac66b92d4c87e;p=git 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 --- diff --git a/config.c b/config.c index 92b0cf447f..d349d52f26 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); }