opendev/gerrit
Code Issues Proposed changes

Merge "Tidy up SubmoduleOp constructor"

Browse Source
This commit is contained in:
Saša Živkov
2015-04-21 08:18:18 +00:00
committed by Gerrit Code Review
parent 53286f2503 1ad61122e2
commit aeb86c3145
1 changed files with 15 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

24
gerrit-server/src/main/java/com/google/gerrit/server/git/SubmoduleOp.java
View File

@@ -95,15 +95,21 @@ public class SubmoduleOp {
private final ChangeHooks changeHooks;
@Inject
public SubmoduleOp(@Assisted final Branch.NameKey destBranch,
@Assisted RevCommit mergeTip, @Assisted RevWalk rw,
@CanonicalWebUrl @Nullable final Provider<String> urlProvider,
final SchemaFactory<ReviewDb> sf, @Assisted Repository db,
@Assisted Project destProject, @Assisted List<Change> submitted,
@Assisted final Map<Change.Id, CodeReviewCommit> commits,
@GerritPersonIdent final PersonIdent myIdent,
GitRepositoryManager repoManager, GitReferenceUpdated gitRefUpdated,
@Nullable @Assisted Account account, ChangeHooks changeHooks) {
public SubmoduleOp(@Assisted Branch.NameKey destBranch,
@Assisted RevCommit mergeTip,
@Assisted RevWalk rw,
@CanonicalWebUrl @Nullable Provider<String> urlProvider,
SchemaFactory<ReviewDb> sf,
@Assisted Repository db,
@Assisted Project destProject,
@Assisted List<Change> submitted,
@Assisted Map<Change.Id,
CodeReviewCommit> commits,
@GerritPersonIdent PersonIdent myIdent,
GitRepositoryManager repoManager,
GitReferenceUpdated gitRefUpdated,
@Nullable @Assisted Account account,
ChangeHooks changeHooks) {
this.destBranch = destBranch;
this.mergeTip = mergeTip;
this.rw = rw;