]> git.ipfire.org Git - thirdparty/git.git/commit - revision.c
Add "--show-all" revision walker flag for debugging
authorLinus Torvalds <torvalds@linux-foundation.org>
Sat, 9 Feb 2008 22:02:07 +0000 (14:02 -0800)
committerJunio C Hamano <gitster@pobox.com>
Wed, 13 Feb 2008 23:59:26 +0000 (15:59 -0800)
commit3131b713013f06285cad3ffdcc61f417ac4ba158
treeb431106812e10a3ab9dab43ec044bfdd48223cf6
parentaa8d53ec387a7baf72ab5e3a91c35bb5bf20eb4d
Add "--show-all" revision walker flag for debugging

It's really not very easy to visualize the commit walker, because - on
purpose - it obvously doesn't show the uninteresting commits!

This adds a "--show-all" flag to the revision walker, which will make
it show uninteresting commits too, and they'll have a '^' in front of
them (it also fixes a logic error for !verbose_header for boundary
commits - we should show the '-' even if left_right isn't shown).

A separate patch to gitk to teach it the new '^' was sent
to paulus.  With the change in place, it actually is interesting
even for the cases that git doesn't have any problems with, ie
for the kernel you can do:

gitk -d --show-all v2.6.24..

and you see just how far down it has to parse things to see it all. The
use of "-d" is a good idea, since the date-ordered toposort is much better
at showing why it goes deep down (ie the date of some of those commits
after 2.6.24 is much older, because they were merged from trees that
weren't rebased).

So I think this is a useful feature even for non-debugging - just to
visualize what git does internally more.

When it actually breaks out due to the "everybody_uninteresting()"
case, it adds the uninteresting commits (both the one it's looking at
now, and the list of pending ones) to the list

This way, we really list *all* the commits we've looked at.

Because we now end up listing commits we may not even have been parsed
at all "show_log" and "show_commit" need to protect against commits
that don't have a commit buffer entry.

That second part is debatable just how it should work. Maybe we shouldn't
show such entries at all (with this patch those entries do get shown, they
just don't get any message shown with them). But I think this is a useful
case.

Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
builtin-rev-list.c
log-tree.c
revision.c
revision.h