Browse Source

Use try-with-resources to fix warnings in AddCommand

- replaces use of deprecated ObjectInserter.release()
- auto-close TreeWalk

Change-Id: I540ee711b8c3430a71fdff07add506b7d9c039dc
Signed-off-by: Matthias Sohn <matthias.sohn@sap.com>
tags/v4.0.0.201505050340-m2
Matthias Sohn 9 years ago
parent
commit
e5b81e49e8
1 changed files with 2 additions and 4 deletions
  1. 2
    4
      org.eclipse.jgit/src/org/eclipse/jgit/api/AddCommand.java

+ 2
- 4
org.eclipse.jgit/src/org/eclipse/jgit/api/AddCommand.java View File

@@ -137,13 +137,12 @@ public class AddCommand extends GitCommand<DirCache> {
if (filepatterns.contains(".")) //$NON-NLS-1$
addAll = true;

ObjectInserter inserter = repo.newObjectInserter();
try {
try (ObjectInserter inserter = repo.newObjectInserter();
final TreeWalk tw = new TreeWalk(repo)) {
dc = repo.lockDirCache();
DirCacheIterator c;

DirCacheBuilder builder = dc.builder();
final TreeWalk tw = new TreeWalk(repo);
tw.addTree(new DirCacheBuildIterator(builder));
if (workingTreeIterator == null)
workingTreeIterator = new FileTreeIterator(repo);
@@ -212,7 +211,6 @@ public class AddCommand extends GitCommand<DirCache> {
throw new JGitInternalException(
JGitText.get().exceptionCaughtDuringExecutionOfAddCommand, e);
} finally {
inserter.release();
if (dc != null)
dc.unlock();
}

Loading…
Cancel
Save