Fix revision log diff setup, avoid unnecessary diff generation
authorLinus Torvalds <torvalds@linux-foundation.org>
Sat, 29 Sep 2007 16:50:39 +0000 (09:50 -0700)
committerJunio C Hamano <gitster@pobox.com>
Sat, 29 Sep 2007 22:42:32 +0000 (15:42 -0700)
commitb7bb760d5ed4881422673d32f869d140221d3564
tree020763d363c0c949a4598e00ae92cdfb6846275c
parentee8245b54a2e4ea8838d28a7bcce2e07ef887a2c
Fix revision log diff setup, avoid unnecessary diff generation

We used to incorrectly start calculating diffs whenever any argument but
'-z' was recognized by the diff options parsing. That was bogus, since not
all arguments result in diffs being needed, so we just waste a lot of time
and effort on calculating diffs that don't matter.

This actually also fixes another bug in "git log". Try this:

git log -C

and notice how it prints an extra empty line in between log entries, even
though it never prints the actual diff (because we didn't ask for any diff
format, so the diff machinery never prints anything).

With this patch, that bogus empty line is gone, because "revs->diff" is
never set.  So this isn't just a "wasted time and effort" issue, it's also
a slight semantic fix.

Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
revision.c