opendev/gerrit
Code Issues Proposed changes

Make InvalidRevisionException a static class of CreateBranch

Browse Source 
CreateBranch is the only place that InvalidRevisionException is
used.

Make it a private static class in GetBranch, and remove the now
unused one from gerrit.common.errors.

Change-Id: I29e412b8f11117a76cb325baab0ef1f08c86786f
This commit is contained in:
David Pursehouse
2016-02-18 18:04:48 +09:00
parent 8e2795f279
commit 87680bb275
2 changed files with 11 additions and 27 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
gerrit-server/src/main/java/com/google/gerrit/server/project/CreateBranch.java
View File

@@ -16,7 +16,6 @@ package com.google.gerrit.server.project;
import com.google.common.collect.Iterables;
import com.google.gerrit.common.ChangeHooks;
import com.google.gerrit.common.errors.InvalidRevisionException;
import com.google.gerrit.extensions.api.projects.BranchInfo;
import com.google.gerrit.extensions.restapi.AuthException;
import com.google.gerrit.extensions.restapi.BadRequestException;
@@ -253,4 +252,15 @@ public class CreateBranch implements RestModifyView<ProjectResource, Input> {
throw new InvalidRevisionException();
}
}
/** Error indicating the revision is invalid as supplied. */
private static class InvalidRevisionException extends Exception {
private static final long serialVersionUID = 1L;
public static final String MESSAGE = "Invalid Revision";
public InvalidRevisionException() {
super(MESSAGE);
}
}
}