threads / patch / 17558

patchTeach tree iterators to iterate from the workspace root

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

## tl;dr

One message between Feb 4, 2009 and Feb 4, 2009. Diffs are folded; open one to read it.

replies: 0people: 1as markdown or json

Tor Arne Vestbø· Feb 4, 2009, 15:36 UTC · lore

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ø <torarnv@gmail.com>
---
 .../spearce/egit/core/ContainerTreeIterator.java   |   26 ++++++++++++++++++-
 .../spearce/jgit/treewalk/WorkingTreeIterator.java |    2 +-
 2 files changed, 25 insertions(+), 3 deletions(-)
Show changes to 2 files +25 −3

org.spearce.egit.core/src/org/spearce/egit/core/ContainerTreeIterator.java, org.spearce.jgit/src/org/spearce/jgit/treewalk/WorkingTreeIterator.java

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

← back to recent threads