Signed-off-by: Junio C Hamano <junkio@xxxxxxx> --- Linus Torvalds <torvalds@xxxxxxxx> writes: > - always reset the color _before_ printing out the newline. > > This is actually important. You (and Johannes) didn't see it, because > it only matters if you set the background, but if you don't do this, > you get some random and funky behaviour if you pick a color with a > non-default background (which still potentially has problems with tabs > etc, but less so). Doh. I think you did not see it until you tried "git diff" with a stat-dirty but otherwise unmodified file. diff.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/diff.c b/diff.c index 33c8c57..549f4e0 100644 --- a/diff.c +++ b/diff.c @@ -681,7 +681,7 @@ static void builtin_diff(const char *nam const char *lbl[2]; char *a_one, *b_two; const char *set = get_color(o->color_diff, DIFF_METAINFO); - const char *reset = get_color(o->color_diff, DIFF_PLAIN); + const char *reset = get_color(o->color_diff, DIFF_RESET); a_one = quote_two("a/", name_a); b_two = quote_two("b/", name_b); -- 1.4.1.rc1.ga77b7 - : 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