Re: [PATCH v2 3/4] builtin-status: configurable submodule summary size

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