[JGIT PATCH 2/7] Cleanup RevWalk.parseTree semantics
From: Shawn O. Pearce <hidden>
Date: 2016-06-15 22:45:17
Subsystem:
the rest · Maintainer:
Linus Torvalds
When we call parseAny we may have opened the ObjectLoader for the tree we are going after. If that worked we know the object exists in the repository and is certainly recorded as type "tree" so we do not need to open the object a second time before returning to the caller. Signed-off-by: Shawn O. Pearce <redacted> --- .../src/org/spearce/jgit/revwalk/RevWalk.java | 18 +++++++++++++----- 1 files changed, 13 insertions(+), 5 deletions(-)
diff --git a/org.spearce.jgit/src/org/spearce/jgit/revwalk/RevWalk.java b/org.spearce.jgit/src/org/spearce/jgit/revwalk/RevWalk.java
index 243d9b3..5a29dcf 100644
--- a/org.spearce.jgit/src/org/spearce/jgit/revwalk/RevWalk.java
+++ b/org.spearce.jgit/src/org/spearce/jgit/revwalk/RevWalk.java@@ -639,15 +639,21 @@ public RevTree parseTree(final AnyObjectId id) c = ((RevTag) c).getObject(); parse(c); } - if (c instanceof RevCommit) { - c = ((RevCommit) c).getTree(); - parse(c); - } else if (!(c instanceof RevTree)) + + final RevTree t; + if (c instanceof RevCommit) + t = ((RevCommit) c).getTree(); + else if (!(c instanceof RevTree)) throw new IncorrectObjectTypeException(id.toObjectId(), Constants.TYPE_TREE); - final RevTree t = (RevTree) c; + else + t = (RevTree) c; + + if ((t.flags & PARSED) != 0) + return t; if (db.openObject(t).getType() != Constants.OBJ_TREE) throw new IncorrectObjectTypeException(t, Constants.TYPE_TREE); + t.flags |= PARSED; return t; }
@@ -685,10 +691,12 @@ public RevObject parseAny(final AnyObjectId id) } case Constants.OBJ_TREE: { r = new RevTree(ldr.getId()); + r.flags |= PARSED; break; } case Constants.OBJ_BLOB: { r = new RevBlob(ldr.getId()); + r.flags |= PARSED; break; } case Constants.OBJ_TAG: {
--
1.6.0.1.319.g9f32b