commit 3e8c19e0d2b693fea3caeecede9512a350cd27f3
author Daniel Sun <sunlan@apache.org> Mon Jul 13 02:40:57 2020 +0800
committer Paul King <paulk@asert.com.au> Thu Jul 23 14:45:43 2020 +1000
tree 1aaeec8f8af3c2a36d0a48a677344d7cbf4d7ca0
parent 43ed85d287643343dc7893fcf793bd1ea808dc52

GROOVY-9637: prefactor existing utility

src/main/java/org/apache/groovy/ast/tools/ImmutablePropertyUtils.java

diff --git a/src/main/java/org/apache/groovy/ast/tools/ImmutablePropertyUtils.java b/src/main/java/org/apache/groovy/ast/tools/ImmutablePropertyUtils.java
index 4e2bde0..c89b484 100644
--- a/src/main/java/org/apache/groovy/ast/tools/ImmutablePropertyUtils.java
+++ b/src/main/java/org/apache/groovy/ast/tools/ImmutablePropertyUtils.java
@@ -69,7 +69,15 @@
               This list can by extended by providing "known immutable" classes
               via Immutable.knownImmutableClasses
              */
-    private static Set<String> builtinImmutables = new HashSet<String>(Arrays.asList(
+    private static final Set<String> BUILTIN_IMMUTABLES = new HashSet<String>(Arrays.asList(
+            "boolean",
+            "byte",
+            "char",
+            "double",
+            "float",
+            "int",
+            "long",
+            "short",
             "java.lang.Class",
             "java.lang.Boolean",
             "java.lang.Byte",
@@ -196,7 +204,7 @@
     }
 
     public static boolean isBuiltinImmutable(String typeName) {
-        return builtinImmutables.contains(typeName);
+        return BUILTIN_IMMUTABLES.contains(typeName);
     }
 
     private static boolean hasImmutableAnnotation(Class clazz) {