* [EGIT PATCH 2/4] Use the Git sort order.
2007-04-22 21:35 [EGIT PATCH 0/4] Unit test for plugin and then some Robin Rosenberg
2007-04-22 21:35 ` [EGIT PATCH 1/4] Drop unneeded GitTreeStorage Robin Rosenberg
@ 2007-04-22 21:36 ` Robin Rosenberg
2007-04-23 22:39 ` Robin Rosenberg
2007-04-22 21:36 ` [EGIT PATCH 3/4] Add writeRef method and make writeSymref public Robin Rosenberg
2007-04-22 21:36 ` [EGIT PATCH 4/4] Add a JUnit test project for egit Robin Rosenberg
3 siblings, 1 reply; 8+ messages in thread
From: Robin Rosenberg @ 2007-04-22 21:36 UTC (permalink / raw)
To: spearce; +Cc: git
Do not resort trees when reading into memory. This requires
that a number of routines learn to sort tree
entries as if they end with a '/'. To do that a parameter
must be passed. I do this by passing a NUL or '/' byte as
an extra parameter for the comparison routines.
No noticeable performance difference can be noted. The reason
for this patch is to simply implementation of the git index
and make jgit more git-like.
Signed-off-by: Robin Rosenberg <robin.rosenberg@dewire.com>
---
.../src/org/spearce/egit/core/GitBlobStorage.java | 2 -
.../org/spearce/egit/core/GitMoveDeleteHook.java | 17 ++++-
.../src/org/spearce/egit/core/GitStorage.java | 5 +
.../egit/core/internal/mapping/GitFileHistory.java | 11 ++-
.../core/internal/mapping/GitFileRevision.java | 5 +
.../org/spearce/egit/core/op/TrackOperation.java | 8 +-
.../org/spearce/egit/core/op/UntrackOperation.java | 6 +-
.../spearce/egit/core/project/GitProjectData.java | 10 ++-
.../egit/core/project/RepositoryMapping.java | 2 -
.../org.spearce.jgit--All-Tests.launch | 6 ++
.../src/org/spearce/jgit/lib/MergedTree.java | 27 +++++--
.../src/org/spearce/jgit/lib/ObjectWriter.java | 7 --
.../src/org/spearce/jgit/lib/SubtreeSorter.java | 40 -----------
.../src/org/spearce/jgit/lib/Tree.java | 85 ++++++++++++++++-------
.../src/org/spearce/jgit/lib/TreeEntry.java | 9 ++
.../tst/org/spearce/jgit/lib/T0002_Tree.java | 33 +++++----
.../org/spearce/jgit/lib/T0006_DeepSpeedTest.java | 2 -
17 files changed, 160 insertions(+), 115 deletions(-)
diff --git a/org.spearce.egit.core/src/org/spearce/egit/core/GitBlobStorage.java b/org.spearce.egit.core/src/org/spearce/egit/core/GitBlobStorage.java
index c947e2b..3ab2184 100644
--- a/org.spearce.egit.core/src/org/spearce/egit/core/GitBlobStorage.java
+++ b/org.spearce.egit.core/src/org/spearce/egit/core/GitBlobStorage.java
@@ -59,7 +59,7 @@ public class GitBlobStorage implements IStorage {
else
prefix = "";
String name = prefix + resource.getProjectRelativePath().toString();
- entry = tree.findMember(name);
+ entry = tree.findBlobMember(name);
} catch (IOException e) {
// TODO Auto-generated catch block
e.printStackTrace();
diff --git a/org.spearce.egit.core/src/org/spearce/egit/core/GitMoveDeleteHook.java b/org.spearce.egit.core/src/org/spearce/egit/core/GitMoveDeleteHook.java
index e09ac76..389fc76 100644
--- a/org.spearce.egit.core/src/org/spearce/egit/core/GitMoveDeleteHook.java
+++ b/org.spearce.egit.core/src/org/spearce/egit/core/GitMoveDeleteHook.java
@@ -122,7 +122,7 @@ class GitMoveDeleteHook implements IMoveDeleteHook {
if (s != null && m != null && m.getCacheTree() != null) {
try {
- final TreeEntry e = m.getCacheTree().findMember(s);
+ final TreeEntry e = m.getCacheTree().findTreeMember(s);
if (e != null) {
e.delete();
m.recomputeMerge();
@@ -182,7 +182,10 @@ class GitMoveDeleteHook implements IMoveDeleteHook {
}
try {
- srcEnt = srcMap.getCacheTree().findMember(srcPath);
+ if (src.getType() == IResource.FILE)
+ srcEnt = srcMap.getCacheTree().findBlobMember(srcPath);
+ else
+ srcEnt = srcMap.getCacheTree().findTreeMember(srcPath);
if (srcEnt == null) {
return FINISH_FOR_ME;
}
@@ -199,7 +202,11 @@ class GitMoveDeleteHook implements IMoveDeleteHook {
if (dstPath == null) {
dstTree = dstMap.getCacheTree();
} else {
- final TreeEntry e = dstMap.getCacheTree().findMember(dstPath);
+ final TreeEntry e;
+ if (src.getType() == IResource.FILE)
+ e = dstMap.getCacheTree().findBlobMember(dstPath);
+ else
+ e = dstMap.getCacheTree().findTreeMember(dstPath);
if (e == null) {
dstTree = dstMap.getCacheTree().addTree(dstPath);
} else if (e instanceof Tree) {
@@ -216,7 +223,9 @@ class GitMoveDeleteHook implements IMoveDeleteHook {
// What? Something already exists at the destination? Assume
// Eclipse meant for us to replace the item.
//
- final TreeEntry existing = dstTree.findMember(dstName);
+ TreeEntry existing = dstTree.findTreeMember(dstName);
+ if (existing == null)
+ dstTree.findBlobMember(dstName);
if (existing != null) {
existing.delete();
}
diff --git a/org.spearce.egit.core/src/org/spearce/egit/core/GitStorage.java b/org.spearce.egit.core/src/org/spearce/egit/core/GitStorage.java
index ce4ff71..edef5db 100644
--- a/org.spearce.egit.core/src/org/spearce/egit/core/GitStorage.java
+++ b/org.spearce.egit.core/src/org/spearce/egit/core/GitStorage.java
@@ -62,7 +62,10 @@ public class GitStorage implements IStorage {
else
prefix = "";
String name = prefix + resource.getProjectRelativePath().toString();
- entry = tree.findMember(name);
+ if (resource.getType() == IResource.FILE)
+ entry = tree.findBlobMember(name);
+ else
+ entry = tree.findTreeMember(name);
} catch (IOException e) {
// TODO Auto-generated catch block
e.printStackTrace();
diff --git a/org.spearce.egit.core/src/org/spearce/egit/core/internal/mapping/GitFileHistory.java b/org.spearce.egit.core/src/org/spearce/egit/core/internal/mapping/GitFileHistory.java
index 51109fb..0569862 100644
--- a/org.spearce.egit.core/src/org/spearce/egit/core/internal/mapping/GitFileHistory.java
+++ b/org.spearce.egit.core/src/org/spearce/egit/core/internal/mapping/GitFileHistory.java
@@ -147,7 +147,11 @@ public class GitFileHistory extends FileHistory implements IAdaptable {
ObjectId[] currentResourceHash = new ObjectId[lastResourceHash.length];
Tree t = current.getTree();
for (int i = 0; i < currentResourceHash.length; ++i) {
- TreeEntry m = t.findMember(relativeResourceName[i]);
+ TreeEntry m;
+ if (i == relativeResourceName.length-1 && resource.getType() == IResource.FILE)
+ m = t.findBlobMember(relativeResourceName[i]);
+ else
+ m = t.findTreeMember(relativeResourceName[i]);
if (m != null) {
ObjectId id = m.getId();
currentResourceHash[i] = id;
@@ -251,7 +255,10 @@ public class GitFileHistory extends FileHistory implements IAdaptable {
}
TreeEntry currentEntry = current.getTree();
for (int i=0; i < relativeResourceName.length && currentEntry != null; ++i) {
- ((Tree)currentEntry).findMember(relativeResourceName[i]);
+ if (i == relativeResourceName.length-1 && resource.getType() == IResource.FILE)
+ ((Tree)currentEntry).findBlobMember(relativeResourceName[i]);
+ else
+ ((Tree)currentEntry).findTreeMember(relativeResourceName[i]);
}
if (currentEntry != null)
revisions = new IFileRevision[] { new GitFileRevision(current, resource,
diff --git a/org.spearce.egit.core/src/org/spearce/egit/core/internal/mapping/GitFileRevision.java b/org.spearce.egit.core/src/org/spearce/egit/core/internal/mapping/GitFileRevision.java
index def142f..63d8486 100644
--- a/org.spearce.egit.core/src/org/spearce/egit/core/internal/mapping/GitFileRevision.java
+++ b/org.spearce.egit.core/src/org/spearce/egit/core/internal/mapping/GitFileRevision.java
@@ -156,7 +156,10 @@ public class GitFileRevision extends FileRevision {
else {
prefix = prefix + "/";
String name = prefix + relPath;
- return tree.findMember(name);
+ if (resource.getType() == IResource.FILE)
+ return tree.findBlobMember(name);
+ else
+ return tree.findTreeMember(name);
}
} else
return tree;
diff --git a/org.spearce.egit.core/src/org/spearce/egit/core/op/TrackOperation.java b/org.spearce.egit.core/src/org/spearce/egit/core/op/TrackOperation.java
index 2f4488d..df3dba7 100644
--- a/org.spearce.egit.core/src/org/spearce/egit/core/op/TrackOperation.java
+++ b/org.spearce.egit.core/src/org/spearce/egit/core/op/TrackOperation.java
@@ -137,12 +137,14 @@ public class TrackOperation implements IWorkspaceRunnable {
// Uh, what? Why are we adding a phantom resource? Just say no!
//
} else if (toAdd instanceof IFile) {
- if (!t.exists(path)) {
- t.addFile(path);
+ if (!t.existsTree(path)) {
+ if (!t.existsBlob(path)) {
+ t.addFile(path);
+ }
}
} else if (toAdd instanceof IContainer) {
final IResource[] m = ((IContainer) toAdd).members();
- final TreeEntry e = t.findMember(path);
+ final TreeEntry e = t.findTreeMember(path);
final Tree c;
c = e instanceof Tree ? (Tree) e : e == null ? t.addTree(path)
: null;
diff --git a/org.spearce.egit.core/src/org/spearce/egit/core/op/UntrackOperation.java b/org.spearce.egit.core/src/org/spearce/egit/core/op/UntrackOperation.java
index 1a81192..d477644 100644
--- a/org.spearce.egit.core/src/org/spearce/egit/core/op/UntrackOperation.java
+++ b/org.spearce.egit.core/src/org/spearce/egit/core/op/UntrackOperation.java
@@ -120,7 +120,11 @@ public class UntrackOperation implements IWorkspaceRunnable {
try {
final Tree t = m.getCacheTree();
- final TreeEntry e = t.findMember(s);
+ final TreeEntry e;
+ if (torm.getType() == IResource.FILE)
+ e = t.findBlobMember(s);
+ else
+ e = t.findTreeMember(s);
if (e != null) {
e.delete();
tomerge.put(m, m);
diff --git a/org.spearce.egit.core/src/org/spearce/egit/core/project/GitProjectData.java b/org.spearce.egit.core/src/org/spearce/egit/core/project/GitProjectData.java
index 10cc3a0..0c4683e 100644
--- a/org.spearce.egit.core/src/org/spearce/egit/core/project/GitProjectData.java
+++ b/org.spearce.egit.core/src/org/spearce/egit/core/project/GitProjectData.java
@@ -296,7 +296,10 @@ public class GitProjectData {
if (s != null && m != null && m.getActiveDiff() != null) {
try {
- return m.getActiveDiff().findMember(s);
+ if (r.getType() == IResource.FILE)
+ return m.getActiveDiff().findBlobMember(s);
+ else
+ return m.getActiveDiff().findTreeMember(s);
} catch (IOException ioe) {
throw Activator.error(
CoreText.GitProjectData_lazyResolveFailed, ioe);
@@ -416,7 +419,10 @@ public class GitProjectData {
try {
synchronized (cacheTree) {
final TreeEntry e;
- e = cacheTree.findMember(s);
+ if (r.getType() == IResource.FILE)
+ e = cacheTree.findBlobMember(s);
+ else
+ e = cacheTree.findTreeMember(s);
if (e instanceof FileTreeEntry) {
trace("modified " + r + " -> "
+ e.getFullName());
diff --git a/org.spearce.egit.core/src/org/spearce/egit/core/project/RepositoryMapping.java b/org.spearce.egit.core/src/org/spearce/egit/core/project/RepositoryMapping.java
index df10665..6b9beed 100644
--- a/org.spearce.egit.core/src/org/spearce/egit/core/project/RepositoryMapping.java
+++ b/org.spearce.egit.core/src/org/spearce/egit/core/project/RepositoryMapping.java
@@ -210,7 +210,7 @@ public class RepositoryMapping {
Tree head = getRepository().mapTree(Constants.HEAD);
if (head != null) {
if (getSubset() != null) {
- final TreeEntry e = head.findMember(getSubset());
+ final TreeEntry e = head.findTreeMember(getSubset());
e.detachParent();
head = e instanceof Tree ? (Tree) e : null;
}
diff --git a/org.spearce.jgit/org.spearce.jgit--All-Tests.launch b/org.spearce.jgit/org.spearce.jgit--All-Tests.launch
index bdbfdea..0ff3d23 100644
--- a/org.spearce.jgit/org.spearce.jgit--All-Tests.launch
+++ b/org.spearce.jgit/org.spearce.jgit--All-Tests.launch
@@ -3,11 +3,17 @@
<booleanAttribute key="org.eclipse.jdt.junit.KEEPRUNNING_ATTR" value="false"/>
<stringAttribute key="org.eclipse.jdt.launching.MAIN_TYPE" value=""/>
<stringAttribute key="org.eclipse.jdt.junit.TESTNAME" value=""/>
+<listAttribute key="org.eclipse.debug.core.MAPPED_RESOURCE_TYPES">
+<listEntry value="4"/>
+</listAttribute>
<stringAttribute key="org.eclipse.jdt.launching.PROJECT_ATTR" value="org.spearce.jgit"/>
<listAttribute key="org.eclipse.debug.ui.favoriteGroups">
<listEntry value="org.eclipse.debug.ui.launchGroup.run"/>
<listEntry value="org.eclipse.debug.ui.launchGroup.debug"/>
</listAttribute>
+<listAttribute key="org.eclipse.debug.core.MAPPED_RESOURCE_PATHS">
+<listEntry value="/org.spearce.jgit"/>
+</listAttribute>
<stringAttribute key="org.eclipse.jdt.junit.CONTAINER" value="=org.spearce.jgit"/>
<booleanAttribute key="org.eclipse.debug.core.appendEnvironmentVariables" value="true"/>
<stringAttribute key="org.eclipse.jdt.junit.TEST_KIND" value="org.eclipse.jdt.junit.loader.junit3"/>
diff --git a/org.spearce.jgit/src/org/spearce/jgit/lib/MergedTree.java b/org.spearce.jgit/src/org/spearce/jgit/lib/MergedTree.java
index e5b2967..165adc3 100644
--- a/org.spearce.jgit/src/org/spearce/jgit/lib/MergedTree.java
+++ b/org.spearce.jgit/src/org/spearce/jgit/lib/MergedTree.java
@@ -53,7 +53,7 @@ public class MergedTree {
}
private static final int binarySearch(final TreeEntry[] entries,
- final int width, final byte[] nameUTF8, final int nameStart,
+ final int width, final byte[] nameUTF8, final byte nameUTF8last, final int nameStart,
final int nameEnd) {
if (entries.length == 0)
return -1;
@@ -66,7 +66,7 @@ public class MergedTree {
while (entries[ix] == null)
ix++;
cmp = Tree.compareNames(entries[ix].getNameUTF8(), nameUTF8,
- nameStart, nameEnd);
+ nameStart, nameEnd, TreeEntry.lastChar(entries[ix]), nameUTF8last);
if (cmp < 0)
low = mid + 1;
else if (cmp == 0)
@@ -95,11 +95,11 @@ public class MergedTree {
return isModified(sources);
}
- public TreeEntry[] findMember(final String s) throws IOException {
- return findMember(s.getBytes(Constants.CHARACTER_ENCODING), 0);
+ public TreeEntry[] findMember(final String s, final byte slast) throws IOException {
+ return findMember(s.getBytes(Constants.CHARACTER_ENCODING), slast, 0);
}
- public TreeEntry[] findMember(final byte[] s, final int offset)
+ public TreeEntry[] findMember(final byte[] s, final byte slast, final int offset)
throws IOException {
final int srcCnt = sources.length;
int slash;
@@ -108,7 +108,8 @@ public class MergedTree {
for (slash = offset; slash < s.length && s[slash] != '/'; slash++) {
// search for path component terminator
}
- p = binarySearch(merged, srcCnt, s, offset, slash);
+ byte xlast = slash<s.length ? (byte)'/' : slast;
+ p = binarySearch(merged, srcCnt, s, xlast, offset, slash);
if (p < 0)
return null;
@@ -127,7 +128,15 @@ public class MergedTree {
subs[j] = merged[k] instanceof Tree ? (Tree) merged[k] : null;
subtrees[p] = new MergedTree(subs);
}
- return subtrees[p].findMember(s, slash + 1);
+ return subtrees[p].findMember(s, slast, slash + 1);
+ }
+
+ public TreeEntry[] findBlobMember(String s) throws IOException {
+ return findMember(s,(byte)0);
+ }
+
+ public TreeEntry[] findTreeMember(String s) throws IOException {
+ return findMember(s,(byte)'/');
}
private void matchByName() throws IOException {
@@ -160,6 +169,7 @@ public class MergedTree {
newMerged = new TreeEntry[pos * srcCnt];
for (pos = 0, treeId = 0; done < srcCnt; pos += srcCnt, treeId++) {
byte[] minName = null;
+ int minNameLast = 0;
if ((pos + srcCnt) >= newMerged.length) {
final TreeEntry[] t = new TreeEntry[newMerged.length * 2];
@@ -176,10 +186,11 @@ public class MergedTree {
final TreeEntry thisEntry = ents[ti];
final int cmp = minName == null ? -1 : Tree.compareNames(
- thisEntry.getNameUTF8(), minName);
+ thisEntry.getNameUTF8(), minName, TreeEntry.lastChar(thisEntry), minNameLast);
if (cmp < 0) {
minName = thisEntry.getNameUTF8();
+ minNameLast = TreeEntry.lastChar(thisEntry);
for (int j = srcId - 1; j >= 0; j--) {
if (newMerged[pos + j] != null) {
newMerged[pos + j] = null;
diff --git a/org.spearce.jgit/src/org/spearce/jgit/lib/ObjectWriter.java b/org.spearce.jgit/src/org/spearce/jgit/lib/ObjectWriter.java
index 5722eec..f9f2bde 100644
--- a/org.spearce.jgit/src/org/spearce/jgit/lib/ObjectWriter.java
+++ b/org.spearce.jgit/src/org/spearce/jgit/lib/ObjectWriter.java
@@ -25,7 +25,6 @@ import java.io.IOException;
import java.io.InputStream;
import java.io.OutputStreamWriter;
import java.security.MessageDigest;
-import java.util.Arrays;
import java.util.Iterator;
import java.util.zip.Deflater;
import java.util.zip.DeflaterOutputStream;
@@ -82,12 +81,6 @@ public class ObjectWriter {
public ObjectId writeTree(final Tree t) throws IOException {
final ByteArrayOutputStream o = new ByteArrayOutputStream();
final TreeEntry[] items = t.members();
- for (int k = items.length - 1; k >= 0; k--) {
- if (items[k] instanceof Tree) {
- Arrays.sort(items, SubtreeSorter.INSTANCE);
- break;
- }
- }
for (int k = 0; k < items.length; k++) {
final TreeEntry e = items[k];
final ObjectId id = e.getId();
diff --git a/org.spearce.jgit/src/org/spearce/jgit/lib/SubtreeSorter.java b/org.spearce.jgit/src/org/spearce/jgit/lib/SubtreeSorter.java
deleted file mode 100644
index 2557525..0000000
--- a/org.spearce.jgit/src/org/spearce/jgit/lib/SubtreeSorter.java
+++ /dev/null
@@ -1,40 +0,0 @@
-package org.spearce.jgit.lib;
-
-import java.util.Comparator;
-
-public class SubtreeSorter implements Comparator {
- public static final SubtreeSorter INSTANCE = new SubtreeSorter();
-
- public int compare(final Object o1, final Object o2) {
- final TreeEntry a = (TreeEntry) o1;
- final TreeEntry b = (TreeEntry) o2;
- final byte[] aName = a.getNameUTF8();
- final byte[] bName = b.getNameUTF8();
-
- for (int k = 0; k < aName.length && k < bName.length; k++) {
- if (aName[k] < bName[k])
- return -1;
- else if (aName[k] > bName[k])
- return 1;
- }
-
- if (aName.length < bName.length) {
- if (a instanceof Tree && '/' > bName[aName.length])
- return 1;
- return -1;
- } else if (aName.length == bName.length) {
- if (a instanceof Tree && b instanceof Tree)
- return 0;
- else if (a instanceof Tree)
- return 1;
- else if (b instanceof Tree)
- return -1;
- else
- return 0;
- } else {
- if (b instanceof Tree && aName[bName.length] < '/')
- return -1;
- return 1;
- }
- }
-}
diff --git a/org.spearce.jgit/src/org/spearce/jgit/lib/Tree.java b/org.spearce.jgit/src/org/spearce/jgit/lib/Tree.java
index 65ed400..73a5e2c 100644
--- a/org.spearce.jgit/src/org/spearce/jgit/lib/Tree.java
+++ b/org.spearce.jgit/src/org/spearce/jgit/lib/Tree.java
@@ -17,7 +17,6 @@
package org.spearce.jgit.lib;
import java.io.IOException;
-import java.util.Arrays;
import org.spearce.jgit.errors.CorruptObjectException;
import org.spearce.jgit.errors.EntryExistsException;
@@ -26,13 +25,14 @@ import org.spearce.jgit.errors.MissingObjectException;
public class Tree extends TreeEntry implements Treeish {
public static final TreeEntry[] EMPTY_TREE = {};
- public static final int compareNames(final byte[] a, final byte[] b) {
- return compareNames(a, b, 0, b.length);
+ public static final int compareNames(final byte[] a, final byte[] b, final int lasta,final int lastb) {
+ return compareNames(a, b, 0, b.length, lasta, lastb);
}
public static final int compareNames(final byte[] a, final byte[] nameUTF8,
- final int nameStart, final int nameEnd) {
- for (int j = 0, k = nameStart; j < a.length && k < nameEnd; j++, k++) {
+ final int nameStart, final int nameEnd, final int lasta, int lastb) {
+ int j,k;
+ for (j = 0, k = nameStart; j < a.length && k < nameEnd; j++, k++) {
final int aj = a[j] & 0xff;
final int bk = nameUTF8[k] & 0xff;
if (aj < bk)
@@ -40,6 +40,28 @@ public class Tree extends TreeEntry implements Treeish {
else if (aj > bk)
return 1;
}
+ if (j < a.length) {
+ int aj = a[j]&0xff;
+ if (aj < lastb)
+ return -1;
+ else if (aj > lastb)
+ return 1;
+ else
+ return 0;
+ }
+ if (k < nameEnd) {
+ int bk = nameUTF8[k] & 0xff;
+ if (lasta < bk)
+ return -1;
+ else if (lasta > bk)
+ return 1;
+ else
+ return 0;
+ }
+ if (lasta < lastb)
+ return -1;
+ else if (lasta > lastb)
+ return 1;
final int namelength = nameEnd - nameStart;
if (a.length == namelength)
@@ -60,7 +82,7 @@ public class Tree extends TreeEntry implements Treeish {
}
private static final int binarySearch(final TreeEntry[] entries,
- final byte[] nameUTF8, final int nameStart, final int nameEnd) {
+ final byte[] nameUTF8, final int nameUTF8last, final int nameStart, final int nameEnd) {
if (entries.length == 0)
return -1;
int high = entries.length;
@@ -68,7 +90,7 @@ public class Tree extends TreeEntry implements Treeish {
do {
final int mid = (low + high) / 2;
final int cmp = compareNames(entries[mid].getNameUTF8(), nameUTF8,
- nameStart, nameEnd);
+ nameStart, nameEnd, TreeEntry.lastChar(entries[mid]), nameUTF8last);
if (cmp < 0)
low = mid + 1;
else if (cmp == 0)
@@ -151,7 +173,8 @@ public class Tree extends TreeEntry implements Treeish {
}
ensureLoaded();
- p = binarySearch(contents, s, offset, slash);
+ byte xlast = slash<s.length ? (byte)'/' : 0;
+ p = binarySearch(contents, s, xlast, offset, slash);
if (p >= 0 && slash < s.length && contents[p] instanceof Tree)
return ((Tree) contents[p]).addFile(s, slash + 1);
@@ -184,7 +207,7 @@ public class Tree extends TreeEntry implements Treeish {
}
ensureLoaded();
- p = binarySearch(contents, s, offset, slash);
+ p = binarySearch(contents, s, (byte)'/', offset, slash);
if (p >= 0 && slash < s.length && contents[p] instanceof Tree)
return ((Tree) contents[p]).addTree(s, slash + 1);
@@ -202,7 +225,7 @@ public class Tree extends TreeEntry implements Treeish {
final int p;
ensureLoaded();
- p = binarySearch(contents, e.getNameUTF8(), 0, e.getNameUTF8().length);
+ p = binarySearch(contents, e.getNameUTF8(), TreeEntry.lastChar(e), 0, e.getNameUTF8().length);
if (p < 0) {
e.attachParent(this);
insertEntry(p, e);
@@ -227,7 +250,7 @@ public class Tree extends TreeEntry implements Treeish {
void removeEntry(final TreeEntry e) {
final TreeEntry[] c = contents;
- final int p = binarySearch(c, e.getNameUTF8(), 0,
+ final int p = binarySearch(c, e.getNameUTF8(), TreeEntry.lastChar(e), 0,
e.getNameUTF8().length);
if (p >= 0) {
final TreeEntry[] n = new TreeEntry[c.length - 1];
@@ -257,15 +280,23 @@ public class Tree extends TreeEntry implements Treeish {
return c;
}
- public boolean exists(final String s) throws IOException {
- return findMember(s) != null;
+ public boolean exists(final String s, byte slast) throws IOException {
+ return findMember(s, slast) != null;
}
- public TreeEntry findMember(final String s) throws IOException {
- return findMember(s.getBytes(Constants.CHARACTER_ENCODING), 0);
+ public boolean existsTree(String path) throws IOException {
+ return exists(path,(byte)'/');
}
- public TreeEntry findMember(final byte[] s, final int offset)
+ public boolean existsBlob(String path) throws IOException {
+ return exists(path,(byte)0);
+ }
+
+ public TreeEntry findMember(final String s, byte slast) throws IOException {
+ return findMember(s.getBytes(Constants.CHARACTER_ENCODING), slast, 0);
+ }
+
+ public TreeEntry findMember(final byte[] s, final byte slast, final int offset)
throws IOException {
int slash;
int p;
@@ -275,17 +306,26 @@ public class Tree extends TreeEntry implements Treeish {
}
ensureLoaded();
- p = binarySearch(contents, s, offset, slash);
+ byte xlast = slash<s.length ? (byte)'/' : slast;
+ p = binarySearch(contents, s, xlast, offset, slash);
if (p >= 0) {
final TreeEntry r = contents[p];
if (slash < s.length)
- return r instanceof Tree ? ((Tree) r).findMember(s, slash + 1)
+ return r instanceof Tree ? ((Tree) r).findMember(s, slast, slash + 1)
: null;
return r;
}
return null;
}
+ public TreeEntry findBlobMember(String s) throws IOException {
+ return findMember(s,(byte)0);
+ }
+
+ public TreeEntry findTreeMember(String s) throws IOException {
+ return findMember(s,(byte)'/');
+ }
+
public void accept(final TreeVisitor tv, final int flags)
throws IOException {
final TreeEntry[] c;
@@ -326,7 +366,6 @@ public class Tree extends TreeEntry implements Treeish {
int rawPtr = 0;
TreeEntry[] temp = new TreeEntry[64];
int nextIndex = 0;
- boolean resort = false;
while (rawPtr < raw.length) {
int c = raw[rawPtr++] & 0xff;
@@ -362,7 +401,6 @@ public class Tree extends TreeEntry implements Treeish {
ent = new FileTreeEntry(this, id, name, true);
else if (FileMode.TREE.equals(mode)) {
ent = new Tree(this, id, name);
- resort = true;
} else if (FileMode.SYMLINK.equals(mode))
ent = new SymlinkTreeEntry(this, id, name);
else
@@ -388,12 +426,6 @@ public class Tree extends TreeEntry implements Treeish {
contents = n;
}
- // Resort contents using our internal sorting order. Git sorts
- // subtrees as though their names end in '/' but that's not how
- // we sort them in memory. Its all the fault of the index...
- //
- if (resort)
- Arrays.sort(contents);
}
public String toString() {
@@ -403,4 +435,5 @@ public class Tree extends TreeEntry implements Treeish {
r.append(getFullName());
return r.toString();
}
+
}
\ No newline at end of file
diff --git a/org.spearce.jgit/src/org/spearce/jgit/lib/TreeEntry.java b/org.spearce.jgit/src/org/spearce/jgit/lib/TreeEntry.java
index 084a9f3..1219846 100644
--- a/org.spearce.jgit/src/org/spearce/jgit/lib/TreeEntry.java
+++ b/org.spearce.jgit/src/org/spearce/jgit/lib/TreeEntry.java
@@ -126,10 +126,17 @@ public abstract class TreeEntry implements Comparable {
if (this == o)
return 0;
if (o instanceof TreeEntry)
- return Tree.compareNames(nameUTF8, ((TreeEntry) o).nameUTF8);
+ return Tree.compareNames(nameUTF8, ((TreeEntry) o).nameUTF8, lastChar(this), lastChar((TreeEntry)o));
return -1;
}
+ final public static int lastChar(TreeEntry treeEntry) {
+ if (treeEntry instanceof FileTreeEntry)
+ return '\0';
+ else
+ return '/';
+ }
+
public void accept(final TreeVisitor tv) throws IOException {
accept(tv, 0);
}
diff --git a/org.spearce.jgit/tst/org/spearce/jgit/lib/T0002_Tree.java b/org.spearce.jgit/tst/org/spearce/jgit/lib/T0002_Tree.java
index e757ef8..62df113 100644
--- a/org.spearce.jgit/tst/org/spearce/jgit/lib/T0002_Tree.java
+++ b/org.spearce.jgit/tst/org/spearce/jgit/lib/T0002_Tree.java
@@ -38,7 +38,8 @@ public class T0002_Tree extends RepositoryTestCase {
assertTrue("no id", t.getId() == null);
assertTrue("tree is self", t.getTree() == t);
assertTrue("database is r", t.getRepository() == db);
- assertTrue("no foo child", t.findMember("foo") == null);
+ assertTrue("no foo child", t.findTreeMember("foo") == null);
+ assertTrue("no foo child", t.findBlobMember("foo") == null);
}
public void test002_addFile() throws IOException {
@@ -57,7 +58,7 @@ public class T0002_Tree extends RepositoryTestCase {
assertTrue("no id", f.getId() == null);
assertTrue("is modified", t.isModified());
assertTrue("has no id", t.getId() == null);
- assertTrue("found bob", t.findMember(f.getName()) == f);
+ assertTrue("found bob", t.findBlobMember(f.getName()) == f);
final TreeEntry[] i = t.members();
assertTrue("iterator is not empty", i != null && i.length > 0);
@@ -87,7 +88,7 @@ public class T0002_Tree extends RepositoryTestCase {
assertTrue("tree is self", f.getTree() == f);
assertTrue("parent is modified", t.isModified());
assertTrue("parent has no id", t.getId() == null);
- assertTrue("found bob child", t.findMember(f.getName()) == f);
+ assertTrue("found bob child", t.findTreeMember(f.getName()) == f);
final TreeEntry[] i = t.members();
assertTrue("iterator is not empty", i.length > 0);
@@ -145,14 +146,14 @@ public class T0002_Tree extends RepositoryTestCase {
assertTrue("t not modified.", !t.isModified());
assertEquals("full path of h ok", "f/g/h", h.getFullName());
- assertTrue("Can find h", t.findMember(h.getFullName()) == h);
- assertTrue("Can't find f/z", t.findMember("f/z") == null);
- assertTrue("Can't find y/z", t.findMember("y/z") == null);
+ assertTrue("Can find h", t.findTreeMember(h.getFullName()) == h);
+ assertTrue("Can't find f/z", t.findBlobMember("f/z") == null);
+ assertTrue("Can't find y/z", t.findBlobMember("y/z") == null);
final FileTreeEntry i = h.addFile("i");
assertNotNull(i);
assertEquals("full path of i ok", "f/g/h/i", i.getFullName());
- assertTrue("Can find i", t.findMember(i.getFullName()) == i);
+ assertTrue("Can find i", t.findBlobMember(i.getFullName()) == i);
assertTrue("h modified", h.isModified());
assertTrue("g modified", g.isModified());
assertTrue("f modified", f.isModified());
@@ -190,17 +191,17 @@ public class T0002_Tree extends RepositoryTestCase {
public void test008_SubtreeInternalSorting() throws IOException {
final Tree t = new Tree(db);
- final FileTreeEntry e0 = t.addFile("a-");
- final FileTreeEntry e1 = t.addFile("a-b");
- final Tree e2 = t.addTree("a");
- final FileTreeEntry e3 = t.addFile("a=");
- final FileTreeEntry e4 = t.addFile("a=b");
+ final FileTreeEntry e0 = t.addFile("a-b");
+ final FileTreeEntry e1 = t.addFile("a-");
+ final FileTreeEntry e2 = t.addFile("a=b");
+ final Tree e3 = t.addTree("a");
+ final FileTreeEntry e4 = t.addFile("a=");
final TreeEntry[] ents = t.members();
- assertSame(e2, ents[0]);
+ assertSame(e1, ents[0]);
assertSame(e0, ents[1]);
- assertSame(e1, ents[2]);
- assertSame(e3, ents[3]);
- assertSame(e4, ents[4]);
+ assertSame(e3, ents[2]);
+ assertSame(e4, ents[3]);
+ assertSame(e2, ents[4]);
}
}
diff --git a/org.spearce.jgit/tst/org/spearce/jgit/lib/T0006_DeepSpeedTest.java b/org.spearce.jgit/tst/org/spearce/jgit/lib/T0006_DeepSpeedTest.java
index 2ae9d42..b4ae4b8 100644
--- a/org.spearce.jgit/tst/org/spearce/jgit/lib/T0006_DeepSpeedTest.java
+++ b/org.spearce.jgit/tst/org/spearce/jgit/lib/T0006_DeepSpeedTest.java
@@ -39,7 +39,7 @@ public class T0006_DeepSpeedTest extends SpeedTestBase {
if (parent.size() > 0) {
ObjectId parentId = (ObjectId) parent.get(0);
commit = db.mapCommit(parentId);
- TreeEntry m = commit.getTree().findMember("net/netfilter/nf_queue.c");
+ TreeEntry m = commit.getTree().findBlobMember("net/netfilter/nf_queue.c");
if (m != null)
commit.getCommitId().toString();
++n;
^ permalink raw reply related [flat|nested] 8+ messages in thread* [EGIT PATCH 4/4] Add a JUnit test project for egit.
2007-04-22 21:35 [EGIT PATCH 0/4] Unit test for plugin and then some Robin Rosenberg
` (2 preceding siblings ...)
2007-04-22 21:36 ` [EGIT PATCH 3/4] Add writeRef method and make writeSymref public Robin Rosenberg
@ 2007-04-22 21:36 ` Robin Rosenberg
2007-04-23 22:41 ` Robin Rosenberg
3 siblings, 1 reply; 8+ messages in thread
From: Robin Rosenberg @ 2007-04-22 21:36 UTC (permalink / raw)
To: spearce; +Cc: git
Signed-off-by: Robin Rosenberg <robin.rosenberg@dewire.com>
---
org.spearce.egit.core.test/.classpath | 7
org.spearce.egit.core.test/.project | 28 ++
.../.settings/org.eclipse.jdt.core.prefs | 320 +++++++++++++++++++++++
.../.settings/org.eclipse.jdt.ui.prefs | 10 +
org.spearce.egit.core.test/META-INF/MANIFEST.MF | 19 +
org.spearce.egit.core.test/build.properties | 4
.../org.spearce.egit.core--All-Tests.launch | 39 +++
.../egit/core/internal/mapping/T0002_history.java | 108 ++++++++
.../op/T0001_ConnectProviderOperationTest.java | 118 ++++++++
.../src/org/spearce/egit/core/test/Activator.java | 50 ++++
.../org/spearce/egit/core/test/GitTestCase.java | 46 +++
.../org/spearce/egit/core/test/TestProject.java | 169 ++++++++++++
12 files changed, 918 insertions(+), 0 deletions(-)
diff --git a/org.spearce.egit.core.test/.classpath b/org.spearce.egit.core.test/.classpath
new file mode 100644
index 0000000..751c8f2
--- /dev/null
+++ b/org.spearce.egit.core.test/.classpath
@@ -0,0 +1,7 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<classpath>
+ <classpathentry kind="src" path="src"/>
+ <classpathentry kind="con" path="org.eclipse.jdt.launching.JRE_CONTAINER"/>
+ <classpathentry kind="con" path="org.eclipse.pde.core.requiredPlugins"/>
+ <classpathentry kind="output" path="bin"/>
+</classpath>
diff --git a/org.spearce.egit.core.test/.project b/org.spearce.egit.core.test/.project
new file mode 100644
index 0000000..3b7fb2f
--- /dev/null
+++ b/org.spearce.egit.core.test/.project
@@ -0,0 +1,28 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<projectDescription>
+ <name>org.spearce.egit.core.test</name>
+ <comment></comment>
+ <projects>
+ </projects>
+ <buildSpec>
+ <buildCommand>
+ <name>org.eclipse.jdt.core.javabuilder</name>
+ <arguments>
+ </arguments>
+ </buildCommand>
+ <buildCommand>
+ <name>org.eclipse.pde.ManifestBuilder</name>
+ <arguments>
+ </arguments>
+ </buildCommand>
+ <buildCommand>
+ <name>org.eclipse.pde.SchemaBuilder</name>
+ <arguments>
+ </arguments>
+ </buildCommand>
+ </buildSpec>
+ <natures>
+ <nature>org.eclipse.pde.PluginNature</nature>
+ <nature>org.eclipse.jdt.core.javanature</nature>
+ </natures>
+</projectDescription>
diff --git a/org.spearce.egit.core.test/.settings/org.eclipse.jdt.core.prefs b/org.spearce.egit.core.test/.settings/org.eclipse.jdt.core.prefs
new file mode 100644
index 0000000..16fbddc
--- /dev/null
+++ b/org.spearce.egit.core.test/.settings/org.eclipse.jdt.core.prefs
@@ -0,0 +1,320 @@
+#Mon Jan 29 20:50:25 EST 2007
+eclipse.preferences.version=1
+org.eclipse.jdt.core.compiler.codegen.inlineJsrBytecode=disabled
+org.eclipse.jdt.core.compiler.codegen.targetPlatform=1.4
+org.eclipse.jdt.core.compiler.codegen.unusedLocal=preserve
+org.eclipse.jdt.core.compiler.compliance=1.4
+org.eclipse.jdt.core.compiler.debug.lineNumber=generate
+org.eclipse.jdt.core.compiler.debug.localVariable=generate
+org.eclipse.jdt.core.compiler.debug.sourceFile=generate
+org.eclipse.jdt.core.compiler.doc.comment.support=enabled
+org.eclipse.jdt.core.compiler.problem.annotationSuperInterface=warning
+org.eclipse.jdt.core.compiler.problem.assertIdentifier=error
+org.eclipse.jdt.core.compiler.problem.autoboxing=ignore
+org.eclipse.jdt.core.compiler.problem.deprecation=warning
+org.eclipse.jdt.core.compiler.problem.deprecationInDeprecatedCode=disabled
+org.eclipse.jdt.core.compiler.problem.deprecationWhenOverridingDeprecatedMethod=disabled
+org.eclipse.jdt.core.compiler.problem.discouragedReference=warning
+org.eclipse.jdt.core.compiler.problem.emptyStatement=warning
+org.eclipse.jdt.core.compiler.problem.enumIdentifier=error
+org.eclipse.jdt.core.compiler.problem.fallthroughCase=warning
+org.eclipse.jdt.core.compiler.problem.fieldHiding=warning
+org.eclipse.jdt.core.compiler.problem.finalParameterBound=warning
+org.eclipse.jdt.core.compiler.problem.finallyBlockNotCompletingNormally=error
+org.eclipse.jdt.core.compiler.problem.forbiddenReference=error
+org.eclipse.jdt.core.compiler.problem.hiddenCatchBlock=error
+org.eclipse.jdt.core.compiler.problem.incompatibleNonInheritedInterfaceMethod=warning
+org.eclipse.jdt.core.compiler.problem.incompleteEnumSwitch=ignore
+org.eclipse.jdt.core.compiler.problem.indirectStaticAccess=error
+org.eclipse.jdt.core.compiler.problem.invalidJavadoc=error
+org.eclipse.jdt.core.compiler.problem.invalidJavadocTags=enabled
+org.eclipse.jdt.core.compiler.problem.invalidJavadocTagsDeprecatedRef=enabled
+org.eclipse.jdt.core.compiler.problem.invalidJavadocTagsNotVisibleRef=enabled
+org.eclipse.jdt.core.compiler.problem.invalidJavadocTagsVisibility=private
+org.eclipse.jdt.core.compiler.problem.localVariableHiding=warning
+org.eclipse.jdt.core.compiler.problem.methodWithConstructorName=error
+org.eclipse.jdt.core.compiler.problem.missingDeprecatedAnnotation=ignore
+org.eclipse.jdt.core.compiler.problem.missingJavadocComments=warning
+org.eclipse.jdt.core.compiler.problem.missingJavadocCommentsOverriding=disabled
+org.eclipse.jdt.core.compiler.problem.missingJavadocCommentsVisibility=protected
+org.eclipse.jdt.core.compiler.problem.missingJavadocTags=error
+org.eclipse.jdt.core.compiler.problem.missingJavadocTagsOverriding=disabled
+org.eclipse.jdt.core.compiler.problem.missingJavadocTagsVisibility=private
+org.eclipse.jdt.core.compiler.problem.missingOverrideAnnotation=ignore
+org.eclipse.jdt.core.compiler.problem.missingSerialVersion=warning
+org.eclipse.jdt.core.compiler.problem.noEffectAssignment=error
+org.eclipse.jdt.core.compiler.problem.noImplicitStringConversion=error
+org.eclipse.jdt.core.compiler.problem.nonExternalizedStringLiteral=ignore
+org.eclipse.jdt.core.compiler.problem.nullReference=warning
+org.eclipse.jdt.core.compiler.problem.overridingPackageDefaultMethod=warning
+org.eclipse.jdt.core.compiler.problem.parameterAssignment=ignore
+org.eclipse.jdt.core.compiler.problem.possibleAccidentalBooleanAssignment=error
+org.eclipse.jdt.core.compiler.problem.rawTypeReference=ignore
+org.eclipse.jdt.core.compiler.problem.specialParameterHidingField=disabled
+org.eclipse.jdt.core.compiler.problem.staticAccessReceiver=error
+org.eclipse.jdt.core.compiler.problem.suppressWarnings=enabled
+org.eclipse.jdt.core.compiler.problem.syntheticAccessEmulation=warning
+org.eclipse.jdt.core.compiler.problem.typeParameterHiding=warning
+org.eclipse.jdt.core.compiler.problem.uncheckedTypeOperation=warning
+org.eclipse.jdt.core.compiler.problem.undocumentedEmptyBlock=warning
+org.eclipse.jdt.core.compiler.problem.unhandledWarningToken=warning
+org.eclipse.jdt.core.compiler.problem.unnecessaryElse=warning
+org.eclipse.jdt.core.compiler.problem.unnecessaryTypeCheck=error
+org.eclipse.jdt.core.compiler.problem.unqualifiedFieldAccess=ignore
+org.eclipse.jdt.core.compiler.problem.unusedDeclaredThrownException=error
+org.eclipse.jdt.core.compiler.problem.unusedDeclaredThrownExceptionWhenOverriding=disabled
+org.eclipse.jdt.core.compiler.problem.unusedImport=error
+org.eclipse.jdt.core.compiler.problem.unusedLabel=error
+org.eclipse.jdt.core.compiler.problem.unusedLocal=error
+org.eclipse.jdt.core.compiler.problem.unusedParameter=warning
+org.eclipse.jdt.core.compiler.problem.unusedParameterWhenImplementingAbstract=disabled
+org.eclipse.jdt.core.compiler.problem.unusedParameterWhenOverridingConcrete=disabled
+org.eclipse.jdt.core.compiler.problem.unusedPrivateMember=error
+org.eclipse.jdt.core.compiler.problem.varargsArgumentNeedCast=error
+org.eclipse.jdt.core.compiler.source=1.4
+org.eclipse.jdt.core.formatter.align_type_members_on_columns=false
+org.eclipse.jdt.core.formatter.alignment_for_arguments_in_allocation_expression=16
+org.eclipse.jdt.core.formatter.alignment_for_arguments_in_enum_constant=16
+org.eclipse.jdt.core.formatter.alignment_for_arguments_in_explicit_constructor_call=16
+org.eclipse.jdt.core.formatter.alignment_for_arguments_in_method_invocation=16
+org.eclipse.jdt.core.formatter.alignment_for_arguments_in_qualified_allocation_expression=16
+org.eclipse.jdt.core.formatter.alignment_for_assignment=0
+org.eclipse.jdt.core.formatter.alignment_for_binary_expression=16
+org.eclipse.jdt.core.formatter.alignment_for_compact_if=16
+org.eclipse.jdt.core.formatter.alignment_for_conditional_expression=80
+org.eclipse.jdt.core.formatter.alignment_for_enum_constants=0
+org.eclipse.jdt.core.formatter.alignment_for_expressions_in_array_initializer=16
+org.eclipse.jdt.core.formatter.alignment_for_multiple_fields=16
+org.eclipse.jdt.core.formatter.alignment_for_parameters_in_constructor_declaration=16
+org.eclipse.jdt.core.formatter.alignment_for_parameters_in_method_declaration=16
+org.eclipse.jdt.core.formatter.alignment_for_selector_in_method_invocation=16
+org.eclipse.jdt.core.formatter.alignment_for_superclass_in_type_declaration=16
+org.eclipse.jdt.core.formatter.alignment_for_superinterfaces_in_enum_declaration=16
+org.eclipse.jdt.core.formatter.alignment_for_superinterfaces_in_type_declaration=16
+org.eclipse.jdt.core.formatter.alignment_for_throws_clause_in_constructor_declaration=16
+org.eclipse.jdt.core.formatter.alignment_for_throws_clause_in_method_declaration=16
+org.eclipse.jdt.core.formatter.blank_lines_after_imports=1
+org.eclipse.jdt.core.formatter.blank_lines_after_package=1
+org.eclipse.jdt.core.formatter.blank_lines_before_field=1
+org.eclipse.jdt.core.formatter.blank_lines_before_first_class_body_declaration=0
+org.eclipse.jdt.core.formatter.blank_lines_before_imports=1
+org.eclipse.jdt.core.formatter.blank_lines_before_member_type=1
+org.eclipse.jdt.core.formatter.blank_lines_before_method=1
+org.eclipse.jdt.core.formatter.blank_lines_before_new_chunk=1
+org.eclipse.jdt.core.formatter.blank_lines_before_package=0
+org.eclipse.jdt.core.formatter.blank_lines_between_type_declarations=1
+org.eclipse.jdt.core.formatter.brace_position_for_annotation_type_declaration=end_of_line
+org.eclipse.jdt.core.formatter.brace_position_for_anonymous_type_declaration=end_of_line
+org.eclipse.jdt.core.formatter.brace_position_for_array_initializer=end_of_line
+org.eclipse.jdt.core.formatter.brace_position_for_block=end_of_line
+org.eclipse.jdt.core.formatter.brace_position_for_block_in_case=end_of_line
+org.eclipse.jdt.core.formatter.brace_position_for_constructor_declaration=end_of_line
+org.eclipse.jdt.core.formatter.brace_position_for_enum_constant=end_of_line
+org.eclipse.jdt.core.formatter.brace_position_for_enum_declaration=end_of_line
+org.eclipse.jdt.core.formatter.brace_position_for_method_declaration=end_of_line
+org.eclipse.jdt.core.formatter.brace_position_for_switch=end_of_line
+org.eclipse.jdt.core.formatter.brace_position_for_type_declaration=end_of_line
+org.eclipse.jdt.core.formatter.comment.clear_blank_lines=false
+org.eclipse.jdt.core.formatter.comment.format_comments=true
+org.eclipse.jdt.core.formatter.comment.format_header=false
+org.eclipse.jdt.core.formatter.comment.format_html=true
+org.eclipse.jdt.core.formatter.comment.format_source_code=true
+org.eclipse.jdt.core.formatter.comment.indent_parameter_description=true
+org.eclipse.jdt.core.formatter.comment.indent_root_tags=true
+org.eclipse.jdt.core.formatter.comment.insert_new_line_before_root_tags=insert
+org.eclipse.jdt.core.formatter.comment.insert_new_line_for_parameter=insert
+org.eclipse.jdt.core.formatter.comment.line_length=80
+org.eclipse.jdt.core.formatter.compact_else_if=true
+org.eclipse.jdt.core.formatter.continuation_indentation=2
+org.eclipse.jdt.core.formatter.continuation_indentation_for_array_initializer=2
+org.eclipse.jdt.core.formatter.format_guardian_clause_on_one_line=false
+org.eclipse.jdt.core.formatter.indent_body_declarations_compare_to_annotation_declaration_header=true
+org.eclipse.jdt.core.formatter.indent_body_declarations_compare_to_enum_constant_header=true
+org.eclipse.jdt.core.formatter.indent_body_declarations_compare_to_enum_declaration_header=true
+org.eclipse.jdt.core.formatter.indent_body_declarations_compare_to_type_header=true
+org.eclipse.jdt.core.formatter.indent_breaks_compare_to_cases=true
+org.eclipse.jdt.core.formatter.indent_empty_lines=false
+org.eclipse.jdt.core.formatter.indent_statements_compare_to_block=true
+org.eclipse.jdt.core.formatter.indent_statements_compare_to_body=true
+org.eclipse.jdt.core.formatter.indent_switchstatements_compare_to_cases=true
+org.eclipse.jdt.core.formatter.indent_switchstatements_compare_to_switch=false
+org.eclipse.jdt.core.formatter.indentation.size=4
+org.eclipse.jdt.core.formatter.insert_new_line_after_annotation=insert
+org.eclipse.jdt.core.formatter.insert_new_line_after_opening_brace_in_array_initializer=do not insert
+org.eclipse.jdt.core.formatter.insert_new_line_at_end_of_file_if_missing=do not insert
+org.eclipse.jdt.core.formatter.insert_new_line_before_catch_in_try_statement=do not insert
+org.eclipse.jdt.core.formatter.insert_new_line_before_closing_brace_in_array_initializer=do not insert
+org.eclipse.jdt.core.formatter.insert_new_line_before_else_in_if_statement=do not insert
+org.eclipse.jdt.core.formatter.insert_new_line_before_finally_in_try_statement=do not insert
+org.eclipse.jdt.core.formatter.insert_new_line_before_while_in_do_statement=do not insert
+org.eclipse.jdt.core.formatter.insert_new_line_in_empty_annotation_declaration=insert
+org.eclipse.jdt.core.formatter.insert_new_line_in_empty_anonymous_type_declaration=insert
+org.eclipse.jdt.core.formatter.insert_new_line_in_empty_block=insert
+org.eclipse.jdt.core.formatter.insert_new_line_in_empty_enum_constant=insert
+org.eclipse.jdt.core.formatter.insert_new_line_in_empty_enum_declaration=insert
+org.eclipse.jdt.core.formatter.insert_new_line_in_empty_method_body=insert
+org.eclipse.jdt.core.formatter.insert_new_line_in_empty_type_declaration=insert
+org.eclipse.jdt.core.formatter.insert_space_after_and_in_type_parameter=insert
+org.eclipse.jdt.core.formatter.insert_space_after_assignment_operator=insert
+org.eclipse.jdt.core.formatter.insert_space_after_at_in_annotation=do not insert
+org.eclipse.jdt.core.formatter.insert_space_after_at_in_annotation_type_declaration=do not insert
+org.eclipse.jdt.core.formatter.insert_space_after_binary_operator=insert
+org.eclipse.jdt.core.formatter.insert_space_after_closing_angle_bracket_in_type_arguments=insert
+org.eclipse.jdt.core.formatter.insert_space_after_closing_angle_bracket_in_type_parameters=insert
+org.eclipse.jdt.core.formatter.insert_space_after_closing_brace_in_block=insert
+org.eclipse.jdt.core.formatter.insert_space_after_closing_paren_in_cast=insert
+org.eclipse.jdt.core.formatter.insert_space_after_colon_in_assert=insert
+org.eclipse.jdt.core.formatter.insert_space_after_colon_in_case=insert
+org.eclipse.jdt.core.formatter.insert_space_after_colon_in_conditional=insert
+org.eclipse.jdt.core.formatter.insert_space_after_colon_in_for=insert
+org.eclipse.jdt.core.formatter.insert_space_after_colon_in_labeled_statement=insert
+org.eclipse.jdt.core.formatter.insert_space_after_comma_in_allocation_expression=insert
+org.eclipse.jdt.core.formatter.insert_space_after_comma_in_annotation=insert
+org.eclipse.jdt.core.formatter.insert_space_after_comma_in_array_initializer=insert
+org.eclipse.jdt.core.formatter.insert_space_after_comma_in_constructor_declaration_parameters=insert
+org.eclipse.jdt.core.formatter.insert_space_after_comma_in_constructor_declaration_throws=insert
+org.eclipse.jdt.core.formatter.insert_space_after_comma_in_enum_constant_arguments=insert
+org.eclipse.jdt.core.formatter.insert_space_after_comma_in_enum_declarations=insert
+org.eclipse.jdt.core.formatter.insert_space_after_comma_in_explicitconstructorcall_arguments=insert
+org.eclipse.jdt.core.formatter.insert_space_after_comma_in_for_increments=insert
+org.eclipse.jdt.core.formatter.insert_space_after_comma_in_for_inits=insert
+org.eclipse.jdt.core.formatter.insert_space_after_comma_in_method_declaration_parameters=insert
+org.eclipse.jdt.core.formatter.insert_space_after_comma_in_method_declaration_throws=insert
+org.eclipse.jdt.core.formatter.insert_space_after_comma_in_method_invocation_arguments=insert
+org.eclipse.jdt.core.formatter.insert_space_after_comma_in_multiple_field_declarations=insert
+org.eclipse.jdt.core.formatter.insert_space_after_comma_in_multiple_local_declarations=insert
+org.eclipse.jdt.core.formatter.insert_space_after_comma_in_parameterized_type_reference=insert
+org.eclipse.jdt.core.formatter.insert_space_after_comma_in_superinterfaces=insert
+org.eclipse.jdt.core.formatter.insert_space_after_comma_in_type_arguments=insert
+org.eclipse.jdt.core.formatter.insert_space_after_comma_in_type_parameters=insert
+org.eclipse.jdt.core.formatter.insert_space_after_ellipsis=insert
+org.eclipse.jdt.core.formatter.insert_space_after_opening_angle_bracket_in_parameterized_type_reference=do not insert
+org.eclipse.jdt.core.formatter.insert_space_after_opening_angle_bracket_in_type_arguments=do not insert
+org.eclipse.jdt.core.formatter.insert_space_after_opening_angle_bracket_in_type_parameters=do not insert
+org.eclipse.jdt.core.formatter.insert_space_after_opening_brace_in_array_initializer=insert
+org.eclipse.jdt.core.formatter.insert_space_after_opening_bracket_in_array_allocation_expression=do not insert
+org.eclipse.jdt.core.formatter.insert_space_after_opening_bracket_in_array_reference=do not insert
+org.eclipse.jdt.core.formatter.insert_space_after_opening_paren_in_annotation=do not insert
+org.eclipse.jdt.core.formatter.insert_space_after_opening_paren_in_cast=do not insert
+org.eclipse.jdt.core.formatter.insert_space_after_opening_paren_in_catch=do not insert
+org.eclipse.jdt.core.formatter.insert_space_after_opening_paren_in_constructor_declaration=do not insert
+org.eclipse.jdt.core.formatter.insert_space_after_opening_paren_in_enum_constant=do not insert
+org.eclipse.jdt.core.formatter.insert_space_after_opening_paren_in_for=do not insert
+org.eclipse.jdt.core.formatter.insert_space_after_opening_paren_in_if=do not insert
+org.eclipse.jdt.core.formatter.insert_space_after_opening_paren_in_method_declaration=do not insert
+org.eclipse.jdt.core.formatter.insert_space_after_opening_paren_in_method_invocation=do not insert
+org.eclipse.jdt.core.formatter.insert_space_after_opening_paren_in_parenthesized_expression=do not insert
+org.eclipse.jdt.core.formatter.insert_space_after_opening_paren_in_switch=do not insert
+org.eclipse.jdt.core.formatter.insert_space_after_opening_paren_in_synchronized=do not insert
+org.eclipse.jdt.core.formatter.insert_space_after_opening_paren_in_while=do not insert
+org.eclipse.jdt.core.formatter.insert_space_after_postfix_operator=do not insert
+org.eclipse.jdt.core.formatter.insert_space_after_prefix_operator=do not insert
+org.eclipse.jdt.core.formatter.insert_space_after_question_in_conditional=insert
+org.eclipse.jdt.core.formatter.insert_space_after_question_in_wildcard=do not insert
+org.eclipse.jdt.core.formatter.insert_space_after_semicolon_in_for=insert
+org.eclipse.jdt.core.formatter.insert_space_after_unary_operator=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_and_in_type_parameter=insert
+org.eclipse.jdt.core.formatter.insert_space_before_assignment_operator=insert
+org.eclipse.jdt.core.formatter.insert_space_before_at_in_annotation_type_declaration=insert
+org.eclipse.jdt.core.formatter.insert_space_before_binary_operator=insert
+org.eclipse.jdt.core.formatter.insert_space_before_closing_angle_bracket_in_parameterized_type_reference=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_closing_angle_bracket_in_type_arguments=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_closing_angle_bracket_in_type_parameters=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_closing_brace_in_array_initializer=insert
+org.eclipse.jdt.core.formatter.insert_space_before_closing_bracket_in_array_allocation_expression=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_closing_bracket_in_array_reference=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_closing_paren_in_annotation=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_closing_paren_in_cast=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_closing_paren_in_catch=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_closing_paren_in_constructor_declaration=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_closing_paren_in_enum_constant=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_closing_paren_in_for=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_closing_paren_in_if=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_closing_paren_in_method_declaration=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_closing_paren_in_method_invocation=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_closing_paren_in_parenthesized_expression=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_closing_paren_in_switch=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_closing_paren_in_synchronized=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_closing_paren_in_while=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_colon_in_assert=insert
+org.eclipse.jdt.core.formatter.insert_space_before_colon_in_case=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_colon_in_conditional=insert
+org.eclipse.jdt.core.formatter.insert_space_before_colon_in_default=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_colon_in_for=insert
+org.eclipse.jdt.core.formatter.insert_space_before_colon_in_labeled_statement=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_comma_in_allocation_expression=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_comma_in_annotation=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_comma_in_array_initializer=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_comma_in_constructor_declaration_parameters=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_comma_in_constructor_declaration_throws=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_comma_in_enum_constant_arguments=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_comma_in_enum_declarations=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_comma_in_explicitconstructorcall_arguments=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_comma_in_for_increments=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_comma_in_for_inits=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_comma_in_method_declaration_parameters=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_comma_in_method_declaration_throws=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_comma_in_method_invocation_arguments=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_comma_in_multiple_field_declarations=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_comma_in_multiple_local_declarations=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_comma_in_parameterized_type_reference=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_comma_in_superinterfaces=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_comma_in_type_arguments=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_comma_in_type_parameters=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_ellipsis=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_opening_angle_bracket_in_parameterized_type_reference=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_opening_angle_bracket_in_type_arguments=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_opening_angle_bracket_in_type_parameters=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_opening_brace_in_annotation_type_declaration=insert
+org.eclipse.jdt.core.formatter.insert_space_before_opening_brace_in_anonymous_type_declaration=insert
+org.eclipse.jdt.core.formatter.insert_space_before_opening_brace_in_array_initializer=insert
+org.eclipse.jdt.core.formatter.insert_space_before_opening_brace_in_block=insert
+org.eclipse.jdt.core.formatter.insert_space_before_opening_brace_in_constructor_declaration=insert
+org.eclipse.jdt.core.formatter.insert_space_before_opening_brace_in_enum_constant=insert
+org.eclipse.jdt.core.formatter.insert_space_before_opening_brace_in_enum_declaration=insert
+org.eclipse.jdt.core.formatter.insert_space_before_opening_brace_in_method_declaration=insert
+org.eclipse.jdt.core.formatter.insert_space_before_opening_brace_in_switch=insert
+org.eclipse.jdt.core.formatter.insert_space_before_opening_brace_in_type_declaration=insert
+org.eclipse.jdt.core.formatter.insert_space_before_opening_bracket_in_array_allocation_expression=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_opening_bracket_in_array_reference=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_opening_bracket_in_array_type_reference=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_opening_paren_in_annotation=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_opening_paren_in_annotation_type_member_declaration=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_opening_paren_in_catch=insert
+org.eclipse.jdt.core.formatter.insert_space_before_opening_paren_in_constructor_declaration=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_opening_paren_in_enum_constant=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_opening_paren_in_for=insert
+org.eclipse.jdt.core.formatter.insert_space_before_opening_paren_in_if=insert
+org.eclipse.jdt.core.formatter.insert_space_before_opening_paren_in_method_declaration=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_opening_paren_in_method_invocation=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_opening_paren_in_parenthesized_expression=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_opening_paren_in_switch=insert
+org.eclipse.jdt.core.formatter.insert_space_before_opening_paren_in_synchronized=insert
+org.eclipse.jdt.core.formatter.insert_space_before_opening_paren_in_while=insert
+org.eclipse.jdt.core.formatter.insert_space_before_parenthesized_expression_in_return=insert
+org.eclipse.jdt.core.formatter.insert_space_before_postfix_operator=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_prefix_operator=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_question_in_conditional=insert
+org.eclipse.jdt.core.formatter.insert_space_before_question_in_wildcard=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_semicolon=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_semicolon_in_for=do not insert
+org.eclipse.jdt.core.formatter.insert_space_before_unary_operator=do not insert
+org.eclipse.jdt.core.formatter.insert_space_between_brackets_in_array_type_reference=do not insert
+org.eclipse.jdt.core.formatter.insert_space_between_empty_braces_in_array_initializer=do not insert
+org.eclipse.jdt.core.formatter.insert_space_between_empty_brackets_in_array_allocation_expression=do not insert
+org.eclipse.jdt.core.formatter.insert_space_between_empty_parens_in_annotation_type_member_declaration=do not insert
+org.eclipse.jdt.core.formatter.insert_space_between_empty_parens_in_constructor_declaration=do not insert
+org.eclipse.jdt.core.formatter.insert_space_between_empty_parens_in_enum_constant=do not insert
+org.eclipse.jdt.core.formatter.insert_space_between_empty_parens_in_method_declaration=do not insert
+org.eclipse.jdt.core.formatter.insert_space_between_empty_parens_in_method_invocation=do not insert
+org.eclipse.jdt.core.formatter.keep_else_statement_on_same_line=false
+org.eclipse.jdt.core.formatter.keep_empty_array_initializer_on_one_line=false
+org.eclipse.jdt.core.formatter.keep_imple_if_on_one_line=false
+org.eclipse.jdt.core.formatter.keep_then_statement_on_same_line=false
+org.eclipse.jdt.core.formatter.lineSplit=80
+org.eclipse.jdt.core.formatter.number_of_blank_lines_at_beginning_of_method_body=0
+org.eclipse.jdt.core.formatter.number_of_empty_lines_to_preserve=1
+org.eclipse.jdt.core.formatter.put_empty_statement_on_new_line=true
+org.eclipse.jdt.core.formatter.tabulation.char=tab
+org.eclipse.jdt.core.formatter.tabulation.size=4
+org.eclipse.jdt.core.formatter.use_tabs_only_for_leading_indentations=false
diff --git a/org.spearce.egit.core.test/.settings/org.eclipse.jdt.ui.prefs b/org.spearce.egit.core.test/.settings/org.eclipse.jdt.ui.prefs
new file mode 100644
index 0000000..273e64b
--- /dev/null
+++ b/org.spearce.egit.core.test/.settings/org.eclipse.jdt.ui.prefs
@@ -0,0 +1,10 @@
+#Mon Jan 29 20:38:19 EST 2007
+eclipse.preferences.version=1
+formatter_profile=_EGit
+formatter_settings_version=10
+internal.default.compliance=user
+org.eclipse.jdt.ui.ignorelowercasenames=true
+org.eclipse.jdt.ui.importorder=java;javax;org;com;
+org.eclipse.jdt.ui.ondemandthreshold=99
+org.eclipse.jdt.ui.staticondemandthreshold=99
+org.eclipse.jdt.ui.text.custom_code_templates=<?xml version\="1.0" encoding\="UTF-8"?><templates/>
diff --git a/org.spearce.egit.core.test/META-INF/MANIFEST.MF b/org.spearce.egit.core.test/META-INF/MANIFEST.MF
new file mode 100644
index 0000000..e201e46
--- /dev/null
+++ b/org.spearce.egit.core.test/META-INF/MANIFEST.MF
@@ -0,0 +1,19 @@
+Manifest-Version: 1.0
+Bundle-ManifestVersion: 2
+Bundle-Name: Test Plug-in
+Bundle-SymbolicName: org.spearce.egit.core.test
+Bundle-Version: 0.2.0
+Bundle-Activator: org.spearce.egit.core.test.Activator
+Require-Bundle: org.eclipse.core.runtime,
+ org.eclipse.team.core,
+ org.junit,
+ org.spearce.egit.core,
+ org.spearce.egit.ui,
+ org.spearce.jgit,
+ org.eclipse.core.filesystem
+Eclipse-LazyStart: true
+Import-Package: org.eclipse.core.resources,
+ org.eclipse.jdt.core,
+ org.eclipse.jdt.junit,
+ org.eclipse.jdt.launching
+Export-Package: org.spearce.egit.core.internal.mapping
diff --git a/org.spearce.egit.core.test/build.properties b/org.spearce.egit.core.test/build.properties
new file mode 100644
index 0000000..34d2e4d
--- /dev/null
+++ b/org.spearce.egit.core.test/build.properties
@@ -0,0 +1,4 @@
+source.. = src/
+output.. = bin/
+bin.includes = META-INF/,\
+ .
diff --git a/org.spearce.egit.core.test/org.spearce.egit.core--All-Tests.launch b/org.spearce.egit.core.test/org.spearce.egit.core--All-Tests.launch
new file mode 100644
index 0000000..f4c9d3c
--- /dev/null
+++ b/org.spearce.egit.core.test/org.spearce.egit.core--All-Tests.launch
@@ -0,0 +1,39 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<launchConfiguration type="org.eclipse.pde.ui.JunitLaunchConfig">
+<stringAttribute key="product" value="org.eclipse.sdk.ide"/>
+<stringAttribute key="bootstrap" value=""/>
+<booleanAttribute key="useProduct" value="true"/>
+<booleanAttribute key="tracing" value="false"/>
+<listAttribute key="org.eclipse.debug.core.MAPPED_RESOURCE_TYPES">
+<listEntry value="4"/>
+</listAttribute>
+<listAttribute key="org.eclipse.debug.ui.favoriteGroups">
+<listEntry value="org.eclipse.debug.ui.launchGroup.run"/>
+<listEntry value="org.eclipse.debug.ui.launchGroup.debug"/>
+</listAttribute>
+<stringAttribute key="org.eclipse.jdt.junit.CONTAINER" value="=org.spearce.egit.core.test"/>
+<booleanAttribute key="automaticValidate" value="false"/>
+<booleanAttribute key="org.eclipse.jdt.junit.KEEPRUNNING_ATTR" value="false"/>
+<booleanAttribute key="automaticAdd" value="true"/>
+<stringAttribute key="checked" value="[NONE]"/>
+<stringAttribute key="location" value="${workspace_loc}/../junit-workspace"/>
+<stringAttribute key="configLocation" value="${workspace_loc}/.metadata/.plugins/org.eclipse.pde.core/pde-junit"/>
+<booleanAttribute key="clearws" value="true"/>
+<stringAttribute key="org.eclipse.jdt.launching.MAIN_TYPE" value=""/>
+<stringAttribute key="pde.version" value="3.3"/>
+<stringAttribute key="org.eclipse.jdt.launching.VM_ARGUMENTS" value="-ea"/>
+<booleanAttribute key="default" value="true"/>
+<booleanAttribute key="clearConfig" value="true"/>
+<booleanAttribute key="org.eclipse.debug.core.appendEnvironmentVariables" value="true"/>
+<stringAttribute key="org.eclipse.jdt.launching.SOURCE_PATH_PROVIDER" value="org.eclipse.pde.ui.workbenchClasspathProvider"/>
+<booleanAttribute key="useDefaultConfigArea" value="false"/>
+<booleanAttribute key="useDefaultConfig" value="true"/>
+<stringAttribute key="org.eclipse.jdt.junit.TESTNAME" value=""/>
+<stringAttribute key="org.eclipse.jdt.launching.PROJECT_ATTR" value="org.spearce.egit.core.test"/>
+<booleanAttribute key="askclear" value="false"/>
+<listAttribute key="org.eclipse.debug.core.MAPPED_RESOURCE_PATHS">
+<listEntry value="/org.spearce.egit.core.test"/>
+</listAttribute>
+<booleanAttribute key="includeOptional" value="true"/>
+<stringAttribute key="org.eclipse.jdt.junit.TEST_KIND" value="org.eclipse.jdt.junit.loader.junit3"/>
+</launchConfiguration>
diff --git a/org.spearce.egit.core.test/src/org/spearce/egit/core/internal/mapping/T0002_history.java b/org.spearce.egit.core.test/src/org/spearce/egit/core/internal/mapping/T0002_history.java
new file mode 100644
index 0000000..aaf9dce
--- /dev/null
+++ b/org.spearce.egit.core.test/src/org/spearce/egit/core/internal/mapping/T0002_history.java
@@ -0,0 +1,108 @@
+/*
+ * Copyright (C) 2006 Robin Rosenberg <robin.rosenberg@dewire.com>
+ *
+ * This library is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Lesser General Public
+ * License, version 2.1, as published by the Free Software Foundation.
+ *
+ * This library is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+ * Lesser General Public License for more details.
+ *
+ * You should have received a copy of the GNU Lesser General Public
+ * License along with this library; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301
+ */
+package org.spearce.egit.core.internal.mapping;
+
+import java.io.File;
+import java.io.FileWriter;
+import java.io.IOException;
+import java.util.Date;
+
+import org.eclipse.core.runtime.NullProgressMonitor;
+import org.eclipse.team.core.RepositoryProvider;
+import org.eclipse.team.core.history.IFileHistory;
+import org.eclipse.team.core.history.IFileHistoryProvider;
+import org.eclipse.team.core.history.IFileRevision;
+import org.spearce.egit.core.GitProvider;
+import org.spearce.egit.core.op.ConnectProviderOperation;
+import org.spearce.egit.core.test.GitTestCase;
+import org.spearce.jgit.lib.Commit;
+import org.spearce.jgit.lib.FileTreeEntry;
+import org.spearce.jgit.lib.ObjectId;
+import org.spearce.jgit.lib.ObjectWriter;
+import org.spearce.jgit.lib.PersonIdent;
+import org.spearce.jgit.lib.Repository;
+import org.spearce.jgit.lib.Tree;
+
+public class T0002_history extends GitTestCase {
+
+ protected static final PersonIdent jauthor;
+
+ protected static final PersonIdent jcommitter;
+
+ static {
+ jauthor = new PersonIdent("J. Author", "jauthor@example.com");
+ jcommitter = new PersonIdent("J. Committer", "jcommitter@example.com");
+ }
+
+ private File workDir;
+ private File gitDir;
+ private Repository thisGit;
+ private Tree tree;
+ private ObjectWriter objectWriter;
+
+ protected void setUp() throws Exception {
+ super.setUp();
+ project.createSourceFolder();
+ gitDir = new File(project.getProject().getWorkspace().getRoot()
+ .getRawLocation().toFile(), ".git");
+ workDir = gitDir.getParentFile();
+ thisGit = new Repository(gitDir);
+ thisGit.create();
+ objectWriter = new ObjectWriter(thisGit);
+
+ tree = new Tree(thisGit);
+ Tree projectTree = tree.addTree("Project-1");
+ File project1_a_txt = createFile("Project-1/A.txt","A.txt - first version\n");
+ addFile(projectTree,project1_a_txt);
+ projectTree.setId(objectWriter.writeTree(projectTree));
+ tree.setId(objectWriter.writeTree(tree));
+ Commit commit = new Commit(thisGit);
+ commit.setAuthor(new PersonIdent(jauthor, new Date(0L)));
+ commit.setCommitter(new PersonIdent(jcommitter, new Date(0L)));
+ commit.setMessage("Foo\n\nMessage");
+ commit.setTree(tree);
+ ObjectId commitId = objectWriter.writeCommit(commit);
+ thisGit.writeRef("refs/heads/master", commitId);
+
+ ConnectProviderOperation operation = new ConnectProviderOperation(
+ project.getProject(), null);
+ operation.run(null);
+ }
+
+ private void addFile(Tree t,File f) throws IOException {
+ ObjectId id = objectWriter.writeBlob(f);
+ t.addEntry(new FileTreeEntry(t,id,f.getName().getBytes("UTF-8"),false));
+ }
+
+ private File createFile(String name, String content) throws IOException {
+ File f = new File(workDir, name);
+ FileWriter fileWriter = new FileWriter(f);
+ fileWriter.write(content);
+ fileWriter.close();
+ return f;
+ }
+
+ public void testShallowHistory() {
+ GitProvider provider = (GitProvider)RepositoryProvider.getProvider(project.project);
+ assertNotNull(provider);
+ IFileHistoryProvider fileHistoryProvider = provider.getFileHistoryProvider();
+ IFileHistory fileHistory = fileHistoryProvider.getFileHistoryFor(project.getProject().getWorkspace().getRoot().findMember("Project-1/A.txt"), IFileHistoryProvider.SINGLE_LINE_OF_DESCENT, new NullProgressMonitor());
+ IFileRevision[] fileRevisions = fileHistory.getFileRevisions();
+ assertEquals(1, fileRevisions.length);
+ assertEquals("e2eadee5e6de7315df91cf03a75a8b2194a69af2", fileRevisions[0].getContentIdentifier());
+ }
+}
diff --git a/org.spearce.egit.core.test/src/org/spearce/egit/core/op/T0001_ConnectProviderOperationTest.java b/org.spearce.egit.core.test/src/org/spearce/egit/core/op/T0001_ConnectProviderOperationTest.java
new file mode 100644
index 0000000..63ba83e
--- /dev/null
+++ b/org.spearce.egit.core.test/src/org/spearce/egit/core/op/T0001_ConnectProviderOperationTest.java
@@ -0,0 +1,118 @@
+package org.spearce.egit.core.op;
+
+import java.io.ByteArrayInputStream;
+import java.io.File;
+import java.io.IOException;
+import java.util.Date;
+import java.util.TimeZone;
+
+import org.eclipse.core.resources.IFile;
+import org.eclipse.core.runtime.CoreException;
+import org.eclipse.core.runtime.IProgressMonitor;
+import org.eclipse.core.runtime.IStatus;
+import org.eclipse.core.runtime.jobs.Job;
+import org.eclipse.team.core.RepositoryProvider;
+import org.spearce.egit.core.test.GitTestCase;
+import org.spearce.jgit.lib.Commit;
+import org.spearce.jgit.lib.FileTreeEntry;
+import org.spearce.jgit.lib.ObjectId;
+import org.spearce.jgit.lib.ObjectWriter;
+import org.spearce.jgit.lib.PersonIdent;
+import org.spearce.jgit.lib.Repository;
+import org.spearce.jgit.lib.Tree;
+
+public class T0001_ConnectProviderOperationTest extends GitTestCase {
+
+ public void testNoRepository() throws CoreException {
+
+ ConnectProviderOperation operation = new ConnectProviderOperation(
+ project.getProject(), null);
+ operation.run(null);
+
+ // We are shared because we declared as shared
+ assertTrue(RepositoryProvider.isShared(project.getProject()));
+ assertTrue(!gitDir.exists());
+ }
+
+ public void testNewRepository() throws CoreException {
+
+ File gitDir = new File(project.getProject().getWorkspace().getRoot()
+ .getRawLocation().toFile(), ".git");
+ ConnectProviderOperation operation = new ConnectProviderOperation(
+ project.getProject(), gitDir);
+ operation.run(null);
+
+ assertTrue(RepositoryProvider.isShared(project.getProject()));
+
+ assertTrue(gitDir.exists());
+ }
+
+ public void testNewUnsharedFile() throws CoreException, IOException,
+ InterruptedException {
+
+ project.createSourceFolder();
+ IFile fileA = project.getProject().getFolder("src").getFile("A.java");
+ String srcA = "class A {\n" + "}\n";
+ fileA.create(new ByteArrayInputStream(srcA.getBytes()), false, null);
+
+ File gitDir = new File(project.getProject().getWorkspace().getRoot()
+ .getRawLocation().toFile(), ".git");
+ Repository thisGit = new Repository(gitDir);
+ thisGit.create();
+ Tree rootTree = new Tree(thisGit);
+ Tree prjTree = rootTree.addTree(project.getProject().getName());
+ Tree srcTree = prjTree.addTree("src");
+ FileTreeEntry entryA = srcTree.addFile("A.java");
+ ObjectWriter writer = new ObjectWriter(thisGit);
+ entryA.setId(writer.writeBlob(fileA.getRawLocation().toFile()));
+ srcTree.setId(writer.writeTree(srcTree));
+ prjTree.setId(writer.writeTree(prjTree));
+ rootTree.setId(writer.writeTree(rootTree));
+ Commit commit = new Commit(thisGit);
+ commit.setTree(rootTree);
+ commit.setAuthor(new PersonIdent("J. Git", "j.git@egit.org", new Date(
+ 1999, 1, 1), TimeZone.getTimeZone("GMT+1")));
+ commit.setCommitter(commit.getAuthor());
+ commit.setMessage("testNewUnsharedFile\n\nJunit tests\n");
+ ObjectId id = writer.writeCommit(commit);
+ thisGit.writeRef("refs/heads/master", id);
+
+ // helper asserts, this is not what we are really testing
+ assertTrue("blob missing", new File(gitDir,
+ "objects/2e/2439c32d01f0ef39644d561945e8f4b2239489").exists());
+
+ assertTrue("tree missing", new File(gitDir,
+ "objects/87/a105cc4bc0a79885d07ec560c3eee49438acf0").exists());
+ assertTrue("tree missing", new File(gitDir,
+ "objects/08/ccc3d91a14d337a45f355d3d63bd97fd5e4db9").exists());
+ assertTrue("tree missing", new File(gitDir,
+ "objects/9d/aeec817090098f05eeca858e3a552d78b0a346").exists());
+
+ assertTrue("commit missing", new File(gitDir,
+ "objects/45/df73fd9abbc2c61620c036948b1157e4d21253").exists());
+
+ ConnectProviderOperation operation = new ConnectProviderOperation(
+ project.getProject(), null);
+ operation.run(null);
+
+ final boolean f[] = new boolean[1];
+ new Job("wait") {
+ protected IStatus run(IProgressMonitor monitor) {
+
+ System.out.println("MyJob");
+ f[0] = true;
+ return null;
+ }
+
+ {
+ setRule(project.getProject());
+ schedule();
+ }
+ };
+ while (!f[0]) {
+ System.out.println("Waiting");
+ Thread.sleep(1000);
+ }
+ System.out.println("DONE");
+ }
+}
diff --git a/org.spearce.egit.core.test/src/org/spearce/egit/core/test/Activator.java b/org.spearce.egit.core.test/src/org/spearce/egit/core/test/Activator.java
new file mode 100644
index 0000000..9807c84
--- /dev/null
+++ b/org.spearce.egit.core.test/src/org/spearce/egit/core/test/Activator.java
@@ -0,0 +1,50 @@
+package org.spearce.egit.core.test;
+
+import org.eclipse.core.runtime.Plugin;
+import org.osgi.framework.BundleContext;
+
+/**
+ * The activator class controls the plug-in life cycle
+ */
+public class Activator extends Plugin {
+
+ // The plug-in ID
+ public static final String PLUGIN_ID = "org.spearce.egit.core.test";
+
+ // The shared instance
+ private static Activator plugin;
+
+ /**
+ * The constructor
+ */
+ public Activator() {
+ }
+
+ /*
+ * (non-Javadoc)
+ * @see org.eclipse.core.runtime.Plugins#start(org.osgi.framework.BundleContext)
+ */
+ public void start(BundleContext context) throws Exception {
+ super.start(context);
+ plugin = this;
+ }
+
+ /*
+ * (non-Javadoc)
+ * @see org.eclipse.core.runtime.Plugin#stop(org.osgi.framework.BundleContext)
+ */
+ public void stop(BundleContext context) throws Exception {
+ plugin = null;
+ super.stop(context);
+ }
+
+ /**
+ * Returns the shared instance
+ *
+ * @return the shared instance
+ */
+ public static Activator getDefault() {
+ return plugin;
+ }
+
+}
diff --git a/org.spearce.egit.core.test/src/org/spearce/egit/core/test/GitTestCase.java b/org.spearce.egit.core.test/src/org/spearce/egit/core/test/GitTestCase.java
new file mode 100644
index 0000000..4e4fbf0
--- /dev/null
+++ b/org.spearce.egit.core.test/src/org/spearce/egit/core/test/GitTestCase.java
@@ -0,0 +1,46 @@
+package org.spearce.egit.core.test;
+
+import java.io.File;
+import java.io.IOException;
+
+import junit.framework.TestCase;
+
+public abstract class GitTestCase extends TestCase {
+
+ protected TestProject project;
+
+ protected File gitDir;
+
+ protected void setUp() throws Exception {
+ super.setUp();
+ project = new TestProject();
+ gitDir = new File(project.getProject().getWorkspace().getRoot()
+ .getRawLocation().toFile(), ".git");
+ rmrf(gitDir);
+ }
+
+ protected void tearDown() throws Exception {
+ super.tearDown();
+ project.dispose();
+ rmrf(gitDir);
+ }
+
+ private void rmrf(File d) throws IOException {
+ if (!d.exists())
+ return;
+
+ File[] files = d.listFiles();
+ if (files != null) {
+ for (int i = 0; i < files.length; ++i) {
+ if (files[i].isDirectory())
+ rmrf(files[i]);
+ else if (!files[i].delete())
+ throw new IOException(files[i] + " in use or undeletable");
+ }
+ }
+ if (!d.delete())
+ throw new IOException(d + " in use or undeletable");
+ assert !d.exists();
+ }
+
+}
diff --git a/org.spearce.egit.core.test/src/org/spearce/egit/core/test/TestProject.java b/org.spearce.egit.core.test/src/org/spearce/egit/core/test/TestProject.java
new file mode 100644
index 0000000..63c0d4b
--- /dev/null
+++ b/org.spearce.egit.core.test/src/org/spearce/egit/core/test/TestProject.java
@@ -0,0 +1,169 @@
+package org.spearce.egit.core.test;
+
+import java.io.IOException;
+import java.net.MalformedURLException;
+import java.net.URL;
+
+import org.eclipse.core.resources.IFolder;
+import org.eclipse.core.resources.IProject;
+import org.eclipse.core.resources.IProjectDescription;
+import org.eclipse.core.resources.IWorkspaceRoot;
+import org.eclipse.core.resources.ResourcesPlugin;
+import org.eclipse.core.runtime.CoreException;
+import org.eclipse.core.runtime.IPath;
+import org.eclipse.core.runtime.IPluginDescriptor;
+import org.eclipse.core.runtime.IPluginRegistry;
+import org.eclipse.core.runtime.Path;
+import org.eclipse.core.runtime.Platform;
+import org.eclipse.jdt.core.IClasspathEntry;
+import org.eclipse.jdt.core.ICompilationUnit;
+import org.eclipse.jdt.core.IJavaProject;
+import org.eclipse.jdt.core.IPackageFragment;
+import org.eclipse.jdt.core.IPackageFragmentRoot;
+import org.eclipse.jdt.core.IType;
+import org.eclipse.jdt.core.JavaCore;
+import org.eclipse.jdt.core.JavaModelException;
+import org.eclipse.jdt.launching.JavaRuntime;
+
+public class TestProject {
+ public IProject project;
+
+ public IJavaProject javaProject;
+
+ private IPackageFragmentRoot sourceFolder;
+
+ public TestProject() throws CoreException {
+ IWorkspaceRoot root = ResourcesPlugin.getWorkspace().getRoot();
+ project = root.getProject("Project-1");
+ project.create(null);
+ project.open(null);
+ javaProject = JavaCore.create(project);
+ IFolder binFolder = createBinFolder();
+ setJavaNature();
+ javaProject.setRawClasspath(new IClasspathEntry[0], null);
+ createOutputFolder(binFolder);
+ addSystemLibraries();
+ }
+
+ public IProject getProject() {
+ return project;
+ }
+
+ public IJavaProject getJavaProject() {
+ return javaProject;
+ }
+
+ public void addJar(String plugin, String jar) throws MalformedURLException,
+ IOException, JavaModelException {
+ Path result = findFileInPlugin(plugin, jar);
+ IClasspathEntry[] oldEntries = javaProject.getRawClasspath();
+ IClasspathEntry[] newEntries = new IClasspathEntry[oldEntries.length + 1];
+ System.arraycopy(oldEntries, 0, newEntries, 0, oldEntries.length);
+ newEntries[oldEntries.length] = JavaCore.newLibraryEntry(result, null,
+ null);
+ javaProject.setRawClasspath(newEntries, null);
+ }
+
+ public IPackageFragment createPackage(String name) throws CoreException {
+ if (sourceFolder == null)
+ sourceFolder = createSourceFolder();
+ return sourceFolder.createPackageFragment(name, false, null);
+ }
+
+ public IType createType(IPackageFragment pack, String cuName, String source)
+ throws JavaModelException {
+ StringBuffer buf = new StringBuffer();
+ buf.append("package " + pack.getElementName() + ";\n");
+ buf.append("\n");
+ buf.append(source);
+ ICompilationUnit cu = pack.createCompilationUnit(cuName,
+ buf.toString(), false, null);
+ return cu.getTypes()[0];
+ }
+
+ public void dispose() throws CoreException {
+ waitForIndexer();
+ project.delete(true, true, null);
+ }
+
+ private IFolder createBinFolder() throws CoreException {
+ IFolder binFolder = project.getFolder("bin");
+ binFolder.create(false, true, null);
+ return binFolder;
+ }
+
+ private void setJavaNature() throws CoreException {
+ IProjectDescription description = project.getDescription();
+ description.setNatureIds(new String[] { JavaCore.NATURE_ID });
+ project.setDescription(description, null);
+ }
+
+ private void createOutputFolder(IFolder binFolder)
+ throws JavaModelException {
+ IPath outputLocation = binFolder.getFullPath();
+ javaProject.setOutputLocation(outputLocation, null);
+ }
+
+ public IPackageFragmentRoot createSourceFolder() throws CoreException {
+ IFolder folder = project.getFolder("src");
+ folder.create(false, true, null);
+ IPackageFragmentRoot root = javaProject.getPackageFragmentRoot(folder);
+ IClasspathEntry[] oldEntries = javaProject.getRawClasspath();
+ IClasspathEntry[] newEntries = new IClasspathEntry[oldEntries.length + 1];
+ System.arraycopy(oldEntries, 0, newEntries, 0, oldEntries.length);
+ newEntries[oldEntries.length] = JavaCore.newSourceEntry(root.getPath());
+ javaProject.setRawClasspath(newEntries, null);
+ return root;
+ }
+
+ private void addSystemLibraries() throws JavaModelException {
+ IClasspathEntry[] oldEntries = javaProject.getRawClasspath();
+ IClasspathEntry[] newEntries = new IClasspathEntry[oldEntries.length + 1];
+ System.arraycopy(oldEntries, 0, newEntries, 0, oldEntries.length);
+ newEntries[oldEntries.length] = JavaRuntime
+ .getDefaultJREContainerEntry();
+ javaProject.setRawClasspath(newEntries, null);
+ }
+
+ private Path findFileInPlugin(String plugin, String file)
+ throws MalformedURLException, IOException {
+ IPluginRegistry registry = Platform.getPluginRegistry();
+ IPluginDescriptor descriptor = registry.getPluginDescriptor(plugin);
+ URL pluginURL = descriptor.getInstallURL();
+ URL jarURL = new URL(pluginURL, file);
+ URL localJarURL = Platform.asLocalURL(jarURL);
+ return new Path(localJarURL.getPath());
+ }
+
+ public void waitForIndexer() {
+ // new SearchEngine().searchAllTypeNames(ResourcesPlugin.getWorkspace(),
+ // null, null, IJavaSearchConstants.EXACT_MATCH,
+ // IJavaSearchConstants.CASE_SENSITIVE,
+ // IJavaSearchConstants.CLASS, SearchEngine
+ // .createJavaSearchScope(new IJavaElement[0]),
+ // new ITypeNameRequestor() {
+ // public void acceptClass(char[] packageName,
+ // char[] simpleTypeName, char[][] enclosingTypeNames,
+ // String path) {
+ // }
+ // public void acceptInterface(char[] packageName,
+ // char[] simpleTypeName, char[][] enclosingTypeNames,
+ // String path) {
+ // }
+ // }, IJavaSearchConstants.WAIT_UNTIL_READY_TO_SEARCH, null);
+ }
+
+ /**
+ * @return Returns the sourceFolder.
+ */
+ public IPackageFragmentRoot getSourceFolder() {
+ return sourceFolder;
+ }
+
+ /**
+ * @param sourceFolder The sourceFolder to set.
+ */
+ public void setSourceFolder(IPackageFragmentRoot sourceFolder) {
+ this.sourceFolder = sourceFolder;
+ }
+}
^ permalink raw reply related [flat|nested] 8+ messages in thread