DORMANTno replies

[JGIT PATCH] Teach tree iterators to iterate from the workspace root

From: Tor Arne Vestbø <hidden>
Date: 2016-06-15 22:46:05
Subsystem: the rest · Maintainer: Linus Torvalds

This is needed to be able to do treewalks of projects that
have their repository in the project's parent directory.

We always filter out '.git' entries when building the entry list,
no matter where we are in the tree, because we assume that the
checks in the ResourceEntry and FileEntry constructors for a
'.git' folder ensure that we never navigate into submodules.

Signed-off-by: Tor Arne Vestbø <redacted>
---
 .../spearce/egit/core/ContainerTreeIterator.java   |   26 ++++++++++++++++++-
 .../spearce/jgit/treewalk/WorkingTreeIterator.java |    2 +-
 2 files changed, 25 insertions(+), 3 deletions(-)
diff --git a/org.spearce.egit.core/src/org/spearce/egit/core/ContainerTreeIterator.java b/org.spearce.egit.core/src/org/spearce/egit/core/ContainerTreeIterator.java
index 61717f5..6d6b72e 100644
--- a/org.spearce.egit.core/src/org/spearce/egit/core/ContainerTreeIterator.java
+++ b/org.spearce.egit.core/src/org/spearce/egit/core/ContainerTreeIterator.java
@@ -15,6 +15,7 @@
 import org.eclipse.core.resources.IContainer;
 import org.eclipse.core.resources.IFile;
 import org.eclipse.core.resources.IResource;
+import org.eclipse.core.resources.IWorkspaceRoot;
 import org.eclipse.core.runtime.CoreException;
 import org.spearce.egit.core.project.RepositoryMapping;
 import org.spearce.jgit.errors.IncorrectObjectTypeException;
@@ -53,7 +54,7 @@ private static String computePrefix(final IContainer base) {
 	private final IContainer node;
 
 	/**
-	 * Construct a new iterator from the workspace.
+	 * Construct a new iterator from a container in the workspace.
 	 * <p>
 	 * The iterator will support traversal over the named container, but only if
 	 * it is contained within a project which has the Git repository provider
@@ -70,6 +71,23 @@ public ContainerTreeIterator(final IContainer base) {
 		init(entries());
 	}
 
+	/**
+	 * Construct a new iterator from the workspace root.
+	 * <p>
+	 * The iterator will support traversal over workspace projects that have
+	 * a Git repository provider connected and is mapped into a Git repository.
+	 * During the iteration the paths will be automatically generated to match
+	 * the proper repository paths for this container's children.
+	 *
+	 * @param root
+	 *            the workspace root to walk over.
+	 */
+	public ContainerTreeIterator(final IWorkspaceRoot root) {
+		super("");
+		node = root;
+		init(entries());
+	}
+
 	private ContainerTreeIterator(final WorkingTreeIterator p,
 			final IContainer base) {
 		super(p);
@@ -119,6 +137,7 @@ ResourceEntry(final IResource f) {
 				else
 					mode = FileMode.REGULAR_FILE;
 				break;
+			case IResource.PROJECT:
 			case IResource.FOLDER: {
 				final IContainer c = (IContainer) f;
 				if (c.findMember(".git") != null)
@@ -140,7 +159,10 @@ public FileMode getMode() {
 
 		@Override
 		public String getName() {
-			return rsrc.getName();
+			if (rsrc.getType() == IResource.PROJECT)
+				return rsrc.getLocation().lastSegment();
+			else
+				return rsrc.getName();
 		}
 
 		@Override
diff --git a/org.spearce.jgit/src/org/spearce/jgit/treewalk/WorkingTreeIterator.java b/org.spearce.jgit/src/org/spearce/jgit/treewalk/WorkingTreeIterator.java
index 6fce150..07ce2ed 100644
--- a/org.spearce.jgit/src/org/spearce/jgit/treewalk/WorkingTreeIterator.java
+++ b/org.spearce.jgit/src/org/spearce/jgit/treewalk/WorkingTreeIterator.java
@@ -324,7 +324,7 @@ protected void init(final Entry[] list) {
 			final String name = e.getName();
 			if (".".equals(name) || "..".equals(name))
 				continue;
-			if (parent == null && ".git".equals(name))
+			if (".git".equals(name))
 				continue;
 			if (i != o)
 				entries[o] = e;
-- 
1.6.1.2.309.g2ea3
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help