Browse Source

InMemoryRepository: Remove unused RevWalk from batch method signature

The RevWalk given in the arguments is not used. According to the
comment at the top of the method, a new RevWalk is intentionally
used in the implementation.

Remove the unused argument.

Change-Id: Iec81a1341d5bf377801475845b96a465753096ef
Signed-off-by: David Pursehouse <david.pursehouse@sonymobile.com>
stable-4.2
David Pursehouse 9 years ago
parent
commit
f04bd20fc6
  1. 4
      org.eclipse.jgit/src/org/eclipse/jgit/internal/storage/dfs/InMemoryRepository.java

4
org.eclipse.jgit/src/org/eclipse/jgit/internal/storage/dfs/InMemoryRepository.java

@ -274,7 +274,7 @@ public class InMemoryRepository extends DfsRepository {
if (performsAtomicTransactions()) { if (performsAtomicTransactions()) {
try { try {
lock.writeLock().lock(); lock.writeLock().lock();
batch(walk, getCommands()); batch(getCommands());
} finally { } finally {
lock.writeLock().unlock(); lock.writeLock().unlock();
} }
@ -304,7 +304,7 @@ public class InMemoryRepository extends DfsRepository {
return new RefCache(ids.toRefList(), sym.toRefList()); return new RefCache(ids.toRefList(), sym.toRefList());
} }
private void batch(RevWalk walk, List<ReceiveCommand> cmds) { private void batch(List<ReceiveCommand> cmds) {
// Validate that the target exists in a new RevWalk, as the RevWalk // Validate that the target exists in a new RevWalk, as the RevWalk
// from the RefUpdate might be reading back unflushed objects. // from the RefUpdate might be reading back unflushed objects.
Map<ObjectId, ObjectId> peeled = new HashMap<>(); Map<ObjectId, ObjectId> peeled = new HashMap<>();

Loading…
Cancel
Save