commit 9fc261c7812f40862e561b2c38683a4aab1d3cfd
author Edwin Kempin <ekempin@google.com> Wed Jan 12 17:23:44 2022 +0100
committer Edwin Kempin <ekempin@google.com> Fri Jan 14 10:13:36 2022 +0100
tree 42d0809ac0be285e33976ac271ab20307d823954
parent 7514e394eb4e6e8c50662364563fa0c080eb0cc4

ProjectConfig#loadPermissionRules: Remove unneeded ImmutableList.copyOf

Permission#getRules() already returns an ImmutableList.

Signed-off-by: Edwin Kempin <ekempin@google.com>
Change-Id: I521873811ec7ab8bad0b0ee3755590cd41641c8f

java/com/google/gerrit/server/project/ProjectConfig.java

diff --git a/java/com/google/gerrit/server/project/ProjectConfig.java b/java/com/google/gerrit/server/project/ProjectConfig.java
index 03886a9..4a17b5c 100644
--- a/java/com/google/gerrit/server/project/ProjectConfig.java
+++ b/java/com/google/gerrit/server/project/ProjectConfig.java
@@ -901,7 +901,7 @@
       Config rc, String section, String subsection, String varName, boolean useRange) {
     Permission.Builder perm = Permission.builder(varName);
     loadPermissionRules(rc, section, subsection, varName, perm, useRange);
-    return ImmutableList.copyOf(perm.build().getRules());
+    return perm.build().getRules();
   }
 
   private void loadPermissionRules(