[PATCH v3 7/7] path.c: remove home_config_paths()
- From
Paul Tan <pyokagan@gmail.com>
- Date
- May 6, 2015, 08:01 UTC
- Message-ID
- <1430899264-18447-6-git-send-email-pyokagan@gmail.com>
- In-Reply-To
- <1430899264-18447-1-git-send-email-pyokagan@gmail.com>
home_config_paths() combines distinct functionality already implemented by expand_user_path() and xdg_config_home(), and it also hard-codes the path ~/.gitconfig, which makes it unsuitable to use for other home config file paths. Since its use will just add unnecessary complexity to the code, remove it.
Signed-off-by: Paul Tan <pyokagan@gmail.com> ---
Re-sending the patch series. There are no changes.
cache.h | 1 - path.c | 28 ---------------------------- 2 files changed, 29 deletions(-)
diff --git a/cache.h b/cache.h index 3512d32..f6970bb 100644 --- a/cache.h +++ b/cache.h @@ -816,7 +816,6 @@ enum scld_error safe_create_leading_directories(char *path); enum scld_error safe_create_leading_directories_const(const char *path); int mkdir_in_gitdir(const char *path); -extern void home_config_paths(char **global, char **xdg, char *file); extern char *expand_user_path(const char *path); const char *enter_repo(const char *path, int strict); static inline int is_absolute_path(const char *path) diff --git a/path.c b/path.c index c28b8f5..6b537cc 100644 --- a/path.c +++ b/path.c @@ -130,34 +130,6 @@ char *git_path(const char *fmt, ...) return ret; } -void home_config_paths(char **global, char **xdg, char *file) -{ - char *xdg_home = getenv("XDG_CONFIG_HOME"); - char *home = getenv("HOME"); - char *to_free = NULL; - - if (!home) { - if (global) - *global = NULL; - } else { - if (!xdg_home) { - to_free = mkpathdup("%s/.config", home); - xdg_home = to_free; - } - if (global) - *global = mkpathdup("%s/.gitconfig", home); - } - - if (xdg) { - if (!xdg_home) - *xdg = NULL; - else - *xdg = mkpathdup("%s/git/%s", xdg_home, file); - } - - free(to_free); -} - char *git_path_submodule(const char *path, const char *fmt, ...) { char *pathname = get_pathname();
-- 2.1.4