Browse Source

TreeWalk: Do not close reader passed explicitly to constructor

The TreeWalk(ObjectReader) constructor is explicitly to handle the case
where the caller is responsible for opening and closing the reader.
The reader should only be closed when it was created in the
TreeWalk(Repository) constructor.

Change-Id: I627681be80d69ea549f953255a64c7b3b68bcec9
stable-4.0
Dave Borowitz 10 years ago
parent
commit
6f4281b11a
  1. 20
      org.eclipse.jgit/src/org/eclipse/jgit/treewalk/TreeWalk.java

20
org.eclipse.jgit/src/org/eclipse/jgit/treewalk/TreeWalk.java

@ -195,6 +195,8 @@ public class TreeWalk implements AutoCloseable {
private final ObjectReader reader;
private final boolean closeReader;
private final MutableObjectId idBuffer = new MutableObjectId();
private TreeFilter filter;
@ -217,22 +219,30 @@ public class TreeWalk implements AutoCloseable {
* Create a new tree walker for a given repository.
*
* @param repo
* the repository the walker will obtain data from.
* the repository the walker will obtain data from. An
* ObjectReader will be created by the walker, and will be closed
* when the walker is closed.
*/
public TreeWalk(final Repository repo) {
this(repo.newObjectReader());
this(repo.newObjectReader(), true);
}
/**
* Create a new tree walker for a given repository.
*
* @param or
* the reader the walker will obtain tree data from.
* the reader the walker will obtain tree data from. The reader
* is not closed when the walker is closed.
*/
public TreeWalk(final ObjectReader or) {
this(or, false);
}
private TreeWalk(final ObjectReader or, final boolean closeReader) {
reader = or;
filter = TreeFilter.ALL;
trees = NO_TREES;
this.closeReader = closeReader;
}
/** @return the reader this walker is using to load objects. */
@ -261,7 +271,9 @@ public class TreeWalk implements AutoCloseable {
*/
@Override
public void close() {
reader.close();
if (closeReader) {
reader.close();
}
}
/**

Loading…
Cancel
Save