revlog: clean up the node of all revision stripped in the C code
authorPierre-Yves David <pierre-yves.david@octobus.net>
Fri, 08 Nov 2019 10:01:10 +0100
changeset 43580 53581e220ba3
parent 43579 dcf9826c8d8c
child 43581 642433629e20
revlog: clean up the node of all revision stripped in the C code For some obscure reason, the loop cleaning up node was skipping the first element… I cannot see a reason for it. The overall code is running fine nevertheless because the node are also explicitly deleted from python. We want to delete this explicit deletion, so we need to fix that code first. This work is part of a refactoring to unify the revlog index and the nodemap. This unification prepare the use of a persistent nodemap. Differential Revision: https://phab.mercurial-scm.org/D7320
mercurial/cext/revlog.c
--- a/mercurial/cext/revlog.c	Sun Nov 03 00:15:12 2019 +0100
+++ b/mercurial/cext/revlog.c	Fri Nov 08 10:01:10 2019 +0100
@@ -2489,7 +2489,7 @@
 		if (self->ntinitialized) {
 			Py_ssize_t i;
 
-			for (i = start + 1; i < self->length; i++) {
+			for (i = start; i < self->length; i++) {
 				const char *node = index_node_existing(self, i);
 				if (node == NULL)
 					return -1;