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 2020/05/30 15:18:36 UTC

[groovy] 07/11: reuse code for put

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 5945c881e2f92d9ba1885c0a5a8bbaf3449a59c6
Author: Eric Milles <er...@thomsonreuters.com>
AuthorDate: Fri May 29 08:49:01 2020 -0500

    reuse code for put
    
    (cherry picked from commit 07b7aa3119d3026cc9c4c220dbd1bb020628892d)
---
 src/main/java/org/codehaus/groovy/ast/NodeMetaDataHandler.java | 9 +--------
 1 file changed, 1 insertion(+), 8 deletions(-)

diff --git a/src/main/java/org/codehaus/groovy/ast/NodeMetaDataHandler.java b/src/main/java/org/codehaus/groovy/ast/NodeMetaDataHandler.java
index 8bfa8f0..5745316 100644
--- a/src/main/java/org/codehaus/groovy/ast/NodeMetaDataHandler.java
+++ b/src/main/java/org/codehaus/groovy/ast/NodeMetaDataHandler.java
@@ -90,14 +90,7 @@ public interface NodeMetaDataHandler {
      *                        data under that key
      */
     default void setNodeMetaData(Object key, Object value) {
-        if (key == null) throw new GroovyBugError("Tried to set meta data with null key on " + this + ".");
-
-        Map metaDataMap = this.getMetaDataMap();
-        if (metaDataMap == null) {
-            metaDataMap = new ListHashMap();
-            this.setMetaDataMap(metaDataMap);
-        }
-        Object old = metaDataMap.put(key, value);
+        Object old = putNodeMetaData(key, value);
         if (old != null) throw new GroovyBugError("Tried to overwrite existing meta data " + this + ".");
     }