diff options
author | Junio C Hamano <gitster@pobox.com> | 2015-05-11 14:23:52 -0700 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2015-05-11 14:23:53 -0700 |
commit | a0c0c2e5c29f54c69fa6735aca6ed3079f5f4190 (patch) | |
tree | cf97c7a2ddad14d59071155d628622f15f0cc803 /builtin | |
parent | Merge branch 'tb/blame-resurrect-convert-to-git' (diff) | |
parent | config: use error() instead of fprintf(stderr, ...) (diff) | |
download | tgif-a0c0c2e5c29f54c69fa6735aca6ed3079f5f4190.tar.xz |
Merge branch 'jn/clean-use-error-not-fprintf-on-stderr'
Some error messages in "git config" were emitted without calling
the usual error() facility.
* jn/clean-use-error-not-fprintf-on-stderr:
config: use error() instead of fprintf(stderr, ...)
Diffstat (limited to 'builtin')
-rw-r--r-- | builtin/config.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/builtin/config.c b/builtin/config.c index bfd3016e83..28f57c8fb9 100644 --- a/builtin/config.c +++ b/builtin/config.c @@ -193,7 +193,7 @@ static int get_value(const char *key_, const char *regex_) key_regexp = (regex_t*)xmalloc(sizeof(regex_t)); if (regcomp(key_regexp, key, REG_EXTENDED)) { - fprintf(stderr, "Invalid key pattern: %s\n", key_); + error("invalid key pattern: %s", key_); free(key_regexp); key_regexp = NULL; ret = CONFIG_INVALID_PATTERN; @@ -214,7 +214,7 @@ static int get_value(const char *key_, const char *regex_) regexp = (regex_t*)xmalloc(sizeof(regex_t)); if (regcomp(regexp, regex_, REG_EXTENDED)) { - fprintf(stderr, "Invalid pattern: %s\n", regex_); + error("invalid pattern: %s", regex_); free(regexp); regexp = NULL; ret = CONFIG_INVALID_PATTERN; |