opendev/gerrit
Code Issues Proposed changes

Remove unnecessary else clause increasing nesting level

Browse Source 
Change-Id: I216ab29b3ded6414dd195ceb1bed40ca849001bf
This commit is contained in:
Matthias Sohn
2020-10-11 00:16:38 +02:00
parent e33a494bad
commit ecef2ac450
1 changed files with 2 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
java/com/google/gerrit/server/submit/ConfiguredSubscriptionGraphFactory.java
View File

@@ -46,9 +46,8 @@ public class ConfiguredSubscriptionGraphFactory implements SubscriptionGraph.Fac
throws SubmoduleConflictException { throws SubmoduleConflictException {
if (cfg.getBoolean("submodule", "enableSuperProjectSubscriptions", true)) { if (cfg.getBoolean("submodule", "enableSuperProjectSubscriptions", true)) {
return subscriptionGraphFactory.compute(updatedBranches, orm); return subscriptionGraphFactory.compute(updatedBranches, orm);
} else {
logger.atFine().log("Updating superprojects disabled");
return SubscriptionGraph.createEmptyGraph(ImmutableSet.copyOf(updatedBranches));
} }
logger.atFine().log("Updating superprojects disabled");
return SubscriptionGraph.createEmptyGraph(ImmutableSet.copyOf(updatedBranches));
} }
} }