From: Linus Torvalds <torvalds@g5.osdl.org>
Date: Wed, 6 Jul 2005 23:52:49 +0000 (-0700)
Subject: git-rev-list: remove the DUPCHECK logic, use SEEN instead
X-Git-Tag: v0.99~41
X-Git-Url: http://git.tremily.us/?a=commitdiff_plain;h=960cea2dd108b425fdc4470b48f5eb02963a9be9;p=git.git

git-rev-list: remove the DUPCHECK logic, use SEEN instead

That's what we should have done in the first place, since it not only
avoids another unnecessary flag, it also protects the commits from
showing up as duplicates later when they show up as parents of another
commit (in the pop_most_recent_commit() path).

This will hopefully also fix --topo-sort.
---

diff --git a/rev-list.c b/rev-list.c
index fb3372476..0dd45129e 100644
--- a/rev-list.c
+++ b/rev-list.c
@@ -9,7 +9,6 @@
 #define INTERESTING	(1u << 1)
 #define COUNTED		(1u << 2)
 #define SHOWN		(1u << 3)
-#define DUPCHECK	(1u << 4)
 
 static const char rev_list_usage[] =
 	"usage: git-rev-list [OPTION] commit-id <commit-id>\n"
@@ -488,9 +487,9 @@ int main(int argc, char **argv)
 		commit = get_commit_reference(arg, flags);
 		if (!commit)
 			continue;
-		if (commit->object.flags & DUPCHECK)
+		if (commit->object.flags & SEEN)
 			continue;
-		commit->object.flags |= DUPCHECK;
+		commit->object.flags |= SEEN;
 		insert(commit, &list);
 	}