Re: [PATCH 3/4] submodule--helper clone: remove double path checking
From: Junio C Hamano <hidden>
Date: 2016-06-15 23:09:08
Stefan Beller [off-list ref] writes:
Just a few lines after the deleted code we call safe_create_leading_directories_const(path + "/.git") so the check is done twice without action in between. Remove the first check.
I am hesitant to call the call to this function a "check". If you
do not yet have the leading directories, they get created.
We make sure that the parent directory of path exists (or create it
otherwise) and then do the same for path + "/.git".
That is equivalent to just making sure that the parent directory of
path + "/.git" exists (or create it otherwise).
Perhaps?
quoted hunk
Signed-off-by: Stefan Beller <redacted> --- builtin/submodule--helper.c | 4 ---- 1 file changed, 4 deletions(-)diff --git a/builtin/submodule--helper.c b/builtin/submodule--helper.c index 88002ca..914e561 100644 --- a/builtin/submodule--helper.c +++ b/builtin/submodule--helper.c@@ -212,11 +212,7 @@ static int module_clone(int argc, const char **argv, const char *prefix) } /* Write a .git file in the submodule to redirect to the superproject. */ - if (safe_create_leading_directories_const(path) < 0) - die(_("could not create directory '%s'"), path); - strbuf_addf(&sb, "%s/.git", path); - if (safe_create_leading_directories_const(sb.buf) < 0) die(_("could not create leading directories of '%s'"), sb.buf); submodule_dot_git = fopen(sb.buf, "w");