This is a pure code move, necessary to kill the global variable git_link
later (and also helps a bit in the next patch).

Signed-off-by: Nguyễn Thái Ngọc Duy <pclo...@gmail.com>
---
 builtin/init-db.c | 50 +++++++++++++++++++++-----------------------------
 1 file changed, 21 insertions(+), 29 deletions(-)

diff --git a/builtin/init-db.c b/builtin/init-db.c
index 5cb05d9..68c1ae3 100644
--- a/builtin/init-db.c
+++ b/builtin/init-db.c
@@ -311,34 +311,6 @@ static void create_object_directory(void)
        strbuf_release(&path);
 }
 
-static int set_git_dir_init(const char *git_dir,
-                           const char *real_git_dir,
-                           int exist_ok)
-{
-       if (real_git_dir) {
-               struct stat st;
-
-               if (!exist_ok && !stat(git_dir, &st))
-                       die(_("%s already exists"), git_dir);
-
-               if (!exist_ok && !stat(real_git_dir, &st))
-                       die(_("%s already exists"), real_git_dir);
-
-               /*
-                * make sure symlinks are resolved because we'll be
-                * moving the target repo later on in separate_git_dir()
-                */
-               git_link = xstrdup(real_path(git_dir));
-               set_git_dir(real_path(real_git_dir));
-       }
-       else {
-               set_git_dir(real_path(git_dir));
-               git_link = NULL;
-       }
-       startup_info->have_repository = 1;
-       return 0;
-}
-
 static void separate_git_dir(const char *git_dir)
 {
        struct stat st;
@@ -364,9 +336,29 @@ int init_db(const char *git_dir, const char *real_git_dir,
            const char *template_dir, unsigned int flags)
 {
        int reinit;
+       int exist_ok = flags & INIT_DB_EXIST_OK;
 
-       set_git_dir_init(git_dir, real_git_dir, flags & INIT_DB_EXIST_OK);
+       if (real_git_dir) {
+               struct stat st;
 
+               if (!exist_ok && !stat(git_dir, &st))
+                       die(_("%s already exists"), git_dir);
+
+               if (!exist_ok && !stat(real_git_dir, &st))
+                       die(_("%s already exists"), real_git_dir);
+
+               /*
+                * make sure symlinks are resolved because we'll be
+                * moving the target repo later on in separate_git_dir()
+                */
+               git_link = xstrdup(real_path(git_dir));
+               set_git_dir(real_path(real_git_dir));
+       }
+       else {
+               set_git_dir(real_path(git_dir));
+               git_link = NULL;
+       }
+       startup_info->have_repository = 1;
        git_dir = get_git_dir();
 
        if (git_link)
-- 
2.8.2.524.g6ff3d78

Reply via email to