You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@groovy.apache.org by su...@apache.org on 2017/12/11 12:22:17 UTC

groovy git commit: Make cached value unmodifiable

Repository: groovy
Updated Branches:
  refs/heads/master 67db12f11 -> 6323cd15f


Make cached value unmodifiable


Project: http://git-wip-us.apache.org/repos/asf/groovy/repo
Commit: http://git-wip-us.apache.org/repos/asf/groovy/commit/6323cd15
Tree: http://git-wip-us.apache.org/repos/asf/groovy/tree/6323cd15
Diff: http://git-wip-us.apache.org/repos/asf/groovy/diff/6323cd15

Branch: refs/heads/master
Commit: 6323cd15f456b8185688a143dbcbba68b216d9f1
Parents: 67db12f
Author: sunlan <su...@apache.org>
Authored: Mon Dec 11 20:22:11 2017 +0800
Committer: sunlan <su...@apache.org>
Committed: Mon Dec 11 20:22:11 2017 +0800

----------------------------------------------------------------------
 .../codehaus/groovy/transform/stc/StaticTypeCheckingSupport.java   | 2 +-
 .../org/codehaus/groovy/macro/transform/MacroMethodsCache.java     | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/groovy/blob/6323cd15/src/main/org/codehaus/groovy/transform/stc/StaticTypeCheckingSupport.java
----------------------------------------------------------------------
diff --git a/src/main/org/codehaus/groovy/transform/stc/StaticTypeCheckingSupport.java b/src/main/org/codehaus/groovy/transform/stc/StaticTypeCheckingSupport.java
index 95919c9..b326061 100644
--- a/src/main/org/codehaus/groovy/transform/stc/StaticTypeCheckingSupport.java
+++ b/src/main/org/codehaus/groovy/transform/stc/StaticTypeCheckingSupport.java
@@ -2189,7 +2189,7 @@ public abstract class StaticTypeCheckingSupport {
                                     );
                             scanner.scanClasspathModules();
 
-                            return getDGMMethods(modules);
+                            return Collections.unmodifiableMap(getDGMMethods(modules));
                         }
                     });
         }

http://git-wip-us.apache.org/repos/asf/groovy/blob/6323cd15/subprojects/groovy-macro/src/main/groovy/org/codehaus/groovy/macro/transform/MacroMethodsCache.java
----------------------------------------------------------------------
diff --git a/subprojects/groovy-macro/src/main/groovy/org/codehaus/groovy/macro/transform/MacroMethodsCache.java b/subprojects/groovy-macro/src/main/groovy/org/codehaus/groovy/macro/transform/MacroMethodsCache.java
index 5a6af4a..01201fd 100644
--- a/subprojects/groovy-macro/src/main/groovy/org/codehaus/groovy/macro/transform/MacroMethodsCache.java
+++ b/subprojects/groovy-macro/src/main/groovy/org/codehaus/groovy/macro/transform/MacroMethodsCache.java
@@ -50,7 +50,7 @@ class MacroMethodsCache {
         return CACHE.getAndPut(classLoader, new EvictableCache.ValueProvider<ClassLoader, Map<String, List<MethodNode>>>() {
             @Override
             public Map<String, List<MethodNode>> provide(ClassLoader key) {
-                return getMacroMethodsFromClassLoader(key);
+                return Collections.unmodifiableMap(getMacroMethodsFromClassLoader(key));
             }
         });
     }