opendev/gerrit
Code Issues Proposed changes

ChangeApi: Remove unnecessarily nested else clause

Browse Source 
Change-Id: I4fe678df9912a087bb0bd2e3ad9e3b209db5b8b2
This commit is contained in:
David Pursehouse
2017-07-26 10:25:40 +01:00
parent 6c1a52b869
commit 3e61b9b465
1 changed files with 1 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
gerrit-gwtui/src/main/java/com/google/gerrit/client/changes/ChangeApi.java
View File

@@ -387,9 +387,8 @@ public class ChangeApi {
public static RestApi change(@Nullable String project, int id) { public static RestApi change(@Nullable String project, int id) {
if (project == null) { if (project == null) {
return new RestApi("/changes/").id(String.valueOf(id)); return new RestApi("/changes/").id(String.valueOf(id));
} else {
return new RestApi("/changes/").id(project, id);
} }
return new RestApi("/changes/").id(project, id);
} }
public static String emptyToNull(String str) { public static String emptyToNull(String str) {