Ping Yin schrieb: > + if (wt_status_submodule_summary == 0) return; Style: if (!wt_status_submodule_summary) return; > struct child_process sm_summary; > + char summary_limit[64]; > + sprintf(summary_limit, "%d", wt_status_submodule_summary); > const char *argv[] = { Declaration after statement. That's not portable. -- Hannes -- 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