diff options
Diffstat (limited to 'java/com/google/gerrit/server/restapi/project/SetAccessUtil.java')
| -rw-r--r-- | java/com/google/gerrit/server/restapi/project/SetAccessUtil.java | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/java/com/google/gerrit/server/restapi/project/SetAccessUtil.java b/java/com/google/gerrit/server/restapi/project/SetAccessUtil.java index 65cc5a23e8..205420ca20 100644 --- a/java/com/google/gerrit/server/restapi/project/SetAccessUtil.java +++ b/java/com/google/gerrit/server/restapi/project/SetAccessUtil.java @@ -14,6 +14,7 @@ package com.google.gerrit.server.restapi.project; +import com.google.common.collect.ImmutableList; import com.google.common.collect.Iterables; import com.google.gerrit.common.data.GlobalCapability; import com.google.gerrit.entities.AccessSection; @@ -41,8 +42,6 @@ import com.google.gerrit.server.project.RefPattern; import com.google.inject.Inject; import com.google.inject.Provider; import com.google.inject.Singleton; -import java.util.ArrayList; -import java.util.Collections; import java.util.List; import java.util.Map; import java.util.Set; @@ -66,13 +65,14 @@ public class SetAccessUtil { this.pluginPermissionsUtil = pluginPermissionsUtil; } - List<AccessSection> getAccessSections(Map<String, AccessSectionInfo> sectionInfos) + ImmutableList<AccessSection> getAccessSections(Map<String, AccessSectionInfo> sectionInfos) throws UnprocessableEntityException { if (sectionInfos == null) { - return Collections.emptyList(); + return ImmutableList.of(); } - List<AccessSection> sections = new ArrayList<>(sectionInfos.size()); + ImmutableList.Builder<AccessSection> sections = + ImmutableList.builderWithExpectedSize(sectionInfos.size()); for (Map.Entry<String, AccessSectionInfo> entry : sectionInfos.entrySet()) { if (entry.getValue().permissions == null) { continue; @@ -120,7 +120,7 @@ public class SetAccessUtil { } sections.add(accessSection.build()); } - return sections; + return sections.build(); } /** |