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/26 00:09:14 UTC

groovy git commit: Use entrySet to iterate map entries

Repository: groovy
Updated Branches:
  refs/heads/master a32df1088 -> 12997ac1d


Use entrySet to iterate map entries


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

Branch: refs/heads/master
Commit: 12997ac1dc4830aeb4a427ce53c89e417b2b5449
Parents: a32df10
Author: sunlan <su...@apache.org>
Authored: Tue Dec 26 08:09:08 2017 +0800
Committer: sunlan <su...@apache.org>
Committed: Tue Dec 26 08:09:08 2017 +0800

----------------------------------------------------------------------
 src/main/groovy/groovy/lang/SpreadMap.java | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/groovy/blob/12997ac1/src/main/groovy/groovy/lang/SpreadMap.java
----------------------------------------------------------------------
diff --git a/src/main/groovy/groovy/lang/SpreadMap.java b/src/main/groovy/groovy/lang/SpreadMap.java
index 7ba9e4f..0fc6542 100644
--- a/src/main/groovy/groovy/lang/SpreadMap.java
+++ b/src/main/groovy/groovy/lang/SpreadMap.java
@@ -68,6 +68,7 @@ public class SpreadMap extends HashMap {
                                    + t + ") cannot be put in this spreadMap.");
     }
 
+    @Override
     public boolean equals(Object that) {
         if (that instanceof SpreadMap) {
             return equals((SpreadMap) that);
@@ -79,8 +80,10 @@ public class SpreadMap extends HashMap {
         if (that == null) return false;        
 
         if (size() == that.size()) {
-            for (Object key : keySet()) {
-                if (!DefaultTypeTransformation.compareEqual(get(key), that.get(key))) {
+            for (Object e : entrySet()) {
+                Map.Entry entry = (Map.Entry) e;
+                Object key = entry.getKey();
+                if (!DefaultTypeTransformation.compareEqual(entry.getValue(), that.get(key))) {
                     return false;
                 }
             }
@@ -89,6 +92,7 @@ public class SpreadMap extends HashMap {
         return false;
     }
 
+    @Override
     public int hashCode() {
         if (hashCode == 0) {
             for (Object key : keySet()) {