cache_name_compare() compares name and stage, nothing else.

Junio C Hamano · Junio C Hamano · commit 7b80be150ce1 · 2006-02-12T23:46:25.000-08:00
The code was a bit unclear in expressing what it wants to compare.

Signed-off-by: Junio C Hamano &lt;junkio@cox.net&gt;
diff --git a/read-cache.c b/read-cache.c
@@ -246,9 +246,9 @@ int cache_name_compare(const char *name1, int flags1, const char *name2, int fla
 	if (len1 > len2)
 		return 1;
 
-	/* Differences between "assume up-to-date" should not matter. */
-	flags1 &= ~CE_VALID;
-	flags2 &= ~CE_VALID;
+	/* Compare stages  */
+	flags1 &= CE_STAGEMASK;
+	flags2 &= CE_STAGEMASK;
 
 	if (flags1 < flags2)
 		return -1;
