diff src/org/tmatesoft/hg/repo/Revlog.java @ 693:32b0d19e8aba

Fix file.isCopy() use for Log/History command. File revisions originating from another file list no parents even in the middle of revision chain
author Artem Tikhomirov <tikhomirov.artem@gmail.com>
date Mon, 05 Aug 2013 12:45:36 +0200
parents 6526d8adbc0f
children 7efabe0cddcf
line wrap: on
line diff
--- a/src/org/tmatesoft/hg/repo/Revlog.java	Sat Aug 03 17:11:33 2013 +0200
+++ b/src/org/tmatesoft/hg/repo/Revlog.java	Mon Aug 05 12:45:36 2013 +0200
@@ -367,6 +367,8 @@
 				if (parentInsp != null) {
 					allRevisions[i] = nid;
 					if (_start > 0) {
+						// there are chances we don't know parents here,
+						// postpone parent dispatching for later, now just collect what's missing
 						firstParentIndexes[i] = parent1RevIndex;
 						secondParentIndexes[i] = parent2RevIndex;
 						if (parent1RevIndex < _start && parent1RevIndex >= 0) {
@@ -376,6 +378,7 @@
 							missingParents.put(parent2RevIndex, null);
 						}
 					} else {
+						// we iterate from the very beginning, got every index we'll need
 						Nodeid p1 = parent1RevIndex == -1 ? Nodeid.NULL : allRevisions[parent1RevIndex];
 						Nodeid p2 = parent2RevIndex == -1 ? Nodeid.NULL : allRevisions[parent2RevIndex];
 						parentInsp.next(revisionIndex, allRevisions[i], parent1RevIndex, parent2RevIndex, p1, p2);
@@ -384,13 +387,16 @@
 				}
 			}
 		});
-		if (parentInsp != null && _start > 0) {
-			assert missingParents.size() > 0; // in fact, more relaxed than assert. rather 'assume'
-			// TODO [post-1.1] int[] IntMap#keys() or even sort of iterator that can modify values
-			for (int k = missingParents.firstKey(), l = missingParents.lastKey(); k <= l; k++) {
-				if (missingParents.containsKey(k)) {
-					Nodeid nid = getRepo().getChangelog().getRevision(k);
-					missingParents.put(k, nid);
+		if (parentInsp != null && _start > 0 ) {
+			if (missingParents.size() > 0) {
+				// it's possible to get empty missingParents when _start > 0 e.g. when n-th file revision
+				// is a copy of another file and hence got -1,-1 parents in this revlog, and we indexWalk(n,n)
+				for (int k = missingParents.firstKey(), l = missingParents.lastKey(); k <= l; k++) {
+					// TODO [post-1.1] int[] IntMap#keys() or even sort of iterator that can modify values
+					if (missingParents.containsKey(k)) {
+						Nodeid nid = getRepo().getChangelog().getRevision(k);
+						missingParents.put(k, nid);
+					}
 				}
 			}
 
@@ -406,6 +412,7 @@
 				} else if (riP1 != -1) {
 					assert riP1 >=0 && riP1 < _start;
 					p1 = missingParents.get(riP1);
+					assert p1 != null;
 				}
 				// same for Pp2
 				if (riP2 >= _start) {
@@ -413,6 +420,7 @@
 				} else if (riP2 != -1) {
 					assert riP2 >= 0 && riP2 < _start;
 					p2 = missingParents.get(riP2);
+					assert p2 != null;
 				}
 				parentInsp.next(revNum, allRevisions[i], riP1, riP2, p1, p2);
 			}