On Sun, Oct 19, 2014 at 01:14:20PM +0200, René Scharfe wrote: > --- a/wt-status.c > +++ b/wt-status.c > @@ -726,14 +726,14 @@ static void wt_status_print_changed(struct wt_status *s) > static void wt_status_print_submodule_summary(struct wt_status *s, int uncommitted) > { > struct child_process sm_summary = CHILD_PROCESS_INIT; > - struct argv_array env = ARGV_ARRAY_INIT; > struct argv_array argv = ARGV_ARRAY_INIT; I don't think it belongs in this patch, but a follow-on 3/2 might be to give argv the same treatment. -Peff -- 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