MergeSuperSet: Remove toWalk variable to improve readability
This makes the handling of visible/non-visible commits consistent with
the handling of visible/non-visible changes.
Change-Id: I97a56211a39a2462ed6ac667d9ffd7dea18ea804
Signed-off-by: Edwin Kempin <ekempin@google.com>
diff --git a/gerrit-server/src/main/java/com/google/gerrit/server/git/MergeSuperSet.java b/gerrit-server/src/main/java/com/google/gerrit/server/git/MergeSuperSet.java
index 303dec3..32dc7bc 100644
--- a/gerrit-server/src/main/java/com/google/gerrit/server/git/MergeSuperSet.java
+++ b/gerrit-server/src/main/java/com/google/gerrit/server/git/MergeSuperSet.java
@@ -242,7 +242,6 @@
// completeChangeSet computation, for example.
visible = false;
}
- Collection<RevCommit> toWalk = visible ? visibleCommits : nonVisibleCommits;
// Pick a revision to use for traversal. If any of the patch sets
// is visible, we use the most recent one. Otherwise, use the current
@@ -276,7 +275,11 @@
// Always include the input, even if merged. This allows
// SubmitStrategyOp to correct the situation later, assuming it gets
// returned by byCommitsOnBranchNotMerged below.
- toWalk.add(commit);
+ if (visible) {
+ visibleCommits.add(commit);
+ } else {
+ nonVisibleCommits.add(commit);
+ }
}
Set<String> visibleHashes =
