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 2023/04/23 15:25:02 UTC

[groovy] branch master updated: Trivial refactoring: simplify code with `replaceAll`

This is an automated email from the ASF dual-hosted git repository.

sunlan pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/groovy.git


The following commit(s) were added to refs/heads/master by this push:
     new c5a74aef1f Trivial refactoring: simplify code with `replaceAll`
c5a74aef1f is described below

commit c5a74aef1f3873fe6305ea600d99855f4bb3ede8
Author: Daniel Sun <su...@apache.org>
AuthorDate: Sun Apr 23 23:24:27 2023 +0800

    Trivial refactoring: simplify code with `replaceAll`
---
 src/main/java/org/codehaus/groovy/control/StaticImportVisitor.java | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/src/main/java/org/codehaus/groovy/control/StaticImportVisitor.java b/src/main/java/org/codehaus/groovy/control/StaticImportVisitor.java
index 0fad93fa3b..52ead33bec 100644
--- a/src/main/java/org/codehaus/groovy/control/StaticImportVisitor.java
+++ b/src/main/java/org/codehaus/groovy/control/StaticImportVisitor.java
@@ -324,9 +324,7 @@ public class StaticImportVisitor extends ClassCodeExpressionTransformer {
                 if (expression instanceof NamedArgumentListExpression) {
                     NamedArgumentListExpression namedArgs = (NamedArgumentListExpression) expression;
                     List<MapEntryExpression> entryExpressions = namedArgs.getMapEntryExpressions();
-                    for (int i = 0; i < entryExpressions.size(); i++) {
-                        entryExpressions.set(i, (MapEntryExpression) transformMapEntryExpression(entryExpressions.get(i), cce.getType()));
-                    }
+                    entryExpressions.replaceAll(me -> (MapEntryExpression) transformMapEntryExpression(me, cce.getType()));
                 }
             }
         }