So, here's a reroll that makes the code cleaner. First patches are cleanups without behavioral changes, only the last one does something new. I'm waiting for more comments to decide what to do with the configuration option. Right now, my preference would be to call it status.oldStyle and default it to false (i.e. change the behavior, but allow old-timers to get back the old one). Matthieu Moy (4): builtin/stripspace.c: fix broken indentation wt-status: use argv_array API get rid of "git submodule summary --for-status" status: introduce status.displayCommentChar to disable display of # Documentation/config.txt | 5 +++ builtin/commit.c | 9 +++++ builtin/stripspace.c | 8 ++--- git-submodule.sh | 17 +-------- t/t7401-submodule-summary.sh | 13 ------- t/t7508-status.sh | 43 +++++++++++++++++++++++ wt-status.c | 82 +++++++++++++++++++++++++++++++++----------- wt-status.h | 1 + 8 files changed, 125 insertions(+), 53 deletions(-) -- 1.8.4.12.gf9d53a3.dirty -- 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