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 2019/11/03 15:42:21 UTC
[groovy] 05/18: Trivial refactoring: Can be replaced with single
'Map.replaceAll' method call
This is an automated email from the ASF dual-hosted git repository.
sunlan pushed a commit to branch GROOVY_3_0_X
in repository https://gitbox.apache.org/repos/asf/groovy.git
commit 3acf0401ad64915d9f9730b2aa4f4d8ad12cce9c
Author: Daniel Sun <su...@apache.org>
AuthorDate: Sun Nov 3 21:32:06 2019 +0800
Trivial refactoring: Can be replaced with single 'Map.replaceAll' method call
(cherry picked from commit 746c05e151764413b180087fc6a4db593c0685c8)
---
.../codehaus/groovy/transform/stc/AbstractExtensionMethodCache.java | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/src/main/java/org/codehaus/groovy/transform/stc/AbstractExtensionMethodCache.java b/src/main/java/org/codehaus/groovy/transform/stc/AbstractExtensionMethodCache.java
index bfb5c04..d65d9d4 100644
--- a/src/main/java/org/codehaus/groovy/transform/stc/AbstractExtensionMethodCache.java
+++ b/src/main/java/org/codehaus/groovy/transform/stc/AbstractExtensionMethodCache.java
@@ -100,9 +100,7 @@ public abstract class AbstractExtensionMethodCache {
}
private Map<String, List<MethodNode>> makeMethodsUnmodifiable(Map<String, List<MethodNode>> methods) {
- for (Map.Entry<String, List<MethodNode>> entry : methods.entrySet()) {
- methods.put(entry.getKey(), Collections.unmodifiableList(entry.getValue()));
- }
+ methods.replaceAll((k, v) -> Collections.unmodifiableList(v));
return Collections.unmodifiableMap(methods);
}