From: Paul Tan <pyokagan@gmail.com>
Date: Wed, 6 May 2015 08:01:02 +0000 (+0800)
Subject: git-commit: replace use of home_config_paths()
X-Git-Tag: v2.4.3~25^2~2
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=e682c9db1a7c6a81bb4c2e8e8c1d5191dae3bd9f;p=git

git-commit: replace use of home_config_paths()

Since home_config_paths() combines two distinct functionality already
implemented by expand_user_path() and xdg_config_home(), and hides the
home config file path ~/.gitconfig. Make the code more explicit by
replacing the use of home_config_paths() with expand_user_path() and
xdg_config_home().

Signed-off-by: Paul Tan <pyokagan@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
---

diff --git a/builtin/commit.c b/builtin/commit.c
index 7d90c35915..b4aaaab5bc 100644
--- a/builtin/commit.c
+++ b/builtin/commit.c
@@ -1402,12 +1402,10 @@ int cmd_status(int argc, const char **argv, const char *prefix)
 
 static const char *implicit_ident_advice(void)
 {
-	char *user_config = NULL;
-	char *xdg_config = NULL;
-	int config_exists;
+	char *user_config = expand_user_path("~/.gitconfig");
+	char *xdg_config = xdg_config_home("config");
+	int config_exists = file_exists(user_config) || file_exists(xdg_config);
 
-	home_config_paths(&user_config, &xdg_config, "config");
-	config_exists = file_exists(user_config) || file_exists(xdg_config);
 	free(user_config);
 	free(xdg_config);