Kong Lucien <Lucien.Kong@xxxxxxxxxxxxxxx> writes:
> + const char *git_dir = getenv(GIT_DIR_ENVIRONMENT);
get_git_dir() would do the same more cleanly I think. And you don't
actually need it because git_path("...") does what you're doing with
mkpath already.
> + it = &(s->change.items[i]);
> + d = it->util;
Isn't it a complex way to say
d = s->change.items[i].util
?
> + status_printf_ln(s, c, _("You are currently editing in a rebase progress."));
I find the wording strange.
"You are currently editing a commit during a rebase"
?
> + wt_status_print_in_progress(s);
> +
I think this deserves several functions (e.g. one for rebase in
progress, one for bisect, ...). The one you have is already rather long,
and may get longer when you start using the advice mechanism.
--
Matthieu Moy
http://www-verimag.imag.fr/~moy/
--
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