opendev/gerrit
Code Issues Proposed changes

SubmitStrategyOp: Remove unused private methods

Browse Source 
Change-Id: I1d1ca3af4f0561631ed0e38a86baded48de6d53e
This commit is contained in:
David Pursehouse
2018-12-20 12:01:39 +09:00
parent 6643e6f420
commit bcf964c0a3
1 changed files with 0 additions and 16 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
java/com/google/gerrit/server/submit/SubmitStrategyOp.java
View File

@@ -21,7 +21,6 @@ import static java.util.Comparator.comparing;
import static java.util.Objects.requireNonNull; import static java.util.Objects.requireNonNull;
import com.google.common.base.Function; import com.google.common.base.Function;
import com.google.common.collect.Iterables;
import com.google.common.flogger.FluentLogger; import com.google.common.flogger.FluentLogger;
import com.google.gerrit.common.data.SubmitRecord; import com.google.gerrit.common.data.SubmitRecord;
import com.google.gerrit.reviewdb.client.Account; import com.google.gerrit.reviewdb.client.Account;
@@ -395,21 +394,6 @@ abstract class SubmitStrategyOp implements BatchUpdateOp {
}; };
} }
private static Iterable<PatchSetApproval> convertPatchSet(
Iterable<PatchSetApproval> approvals, PatchSet.Id psId) {
return Iterables.transform(approvals, convertPatchSet(psId));
}
private static Iterable<PatchSetApproval> zero(Iterable<PatchSetApproval> approvals) {
return Iterables.transform(
approvals,
a -> {
PatchSetApproval copy = new PatchSetApproval(a.getPatchSetId(), a);
copy.setValue((short) 0);
return copy;
});
}
private String getByAccountName() { private String getByAccountName() {
requireNonNull(submitter, "getByAccountName called before submitter populated"); requireNonNull(submitter, "getByAccountName called before submitter populated");
Optional<Account> account = Optional<Account> account =