Matthias Lederhofer <matled@xxxxxxx> writes: > I try to take a closer look at your changes tomorrow evening. Here > are just two short things I saw while taking a short look at the > patch. > > Johannes Schindelin <Johannes.Schindelin@xxxxxx> wrote: >> +const char *get_git_work_tree(void) >> +{ >> + static int initialized = 0; >> + if (!initialized) { >> + work_tree = getenv(GIT_WORK_TREE_ENVIRONMENT); >> + if (!work_tree) { >> + work_tree = git_work_tree_cfg; >> + if (work_tree && !is_absolute_path(work_tree)) >> + work_tree = git_path(work_tree); > > A tab is missing here. I think xstrdup() is missing here, too. - To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html