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

emilles pushed a commit to branch GROOVY-11743
in repository https://gitbox.apache.org/repos/asf/groovy.git

commit 6f73c1f3ae210e94d07b3b48626ef0b8caa6ecbf
Author: Eric Milles <eric.mil...@thomsonreuters.com>
AuthorDate: Tue Aug 26 11:05:07 2025 -0500

    jdk17 items
---
 .../src/main/groovy/org/apache/groovy/gradle/DocGDK.groovy |  2 +-
 .../groovy/transform/ASTTransformationVisitor.java         | 14 ++++++--------
 2 files changed, 7 insertions(+), 9 deletions(-)

diff --git a/build-logic/src/main/groovy/org/apache/groovy/gradle/DocGDK.groovy 
b/build-logic/src/main/groovy/org/apache/groovy/gradle/DocGDK.groovy
index 5487b9682e..b5a00bfd28 100644
--- a/build-logic/src/main/groovy/org/apache/groovy/gradle/DocGDK.groovy
+++ b/build-logic/src/main/groovy/org/apache/groovy/gradle/DocGDK.groovy
@@ -78,7 +78,7 @@ class DocGDK extends DefaultTask {
                      '-link',
                      
'groovy,org.codehaus.groovy,org.apache.groovy=https://docs.groovy-lang.org/latest/html/gapi/',
                      '-link',
-                     
'java,org.xml,javax,org.w3c=https://docs.oracle.com/en/java/javase/11/docs/api/java.base/']
 + classes.get()
+                     
'java,org.xml,javax,org.w3c=https://docs.oracle.com/en/java/javase/17/docs/api/java.base/']
 + classes.get()
             )
         }
         fs.copy {
diff --git 
a/src/main/java/org/codehaus/groovy/transform/ASTTransformationVisitor.java 
b/src/main/java/org/codehaus/groovy/transform/ASTTransformationVisitor.java
index c18a410e9a..7687a87a20 100644
--- a/src/main/java/org/codehaus/groovy/transform/ASTTransformationVisitor.java
+++ b/src/main/java/org/codehaus/groovy/transform/ASTTransformationVisitor.java
@@ -143,15 +143,15 @@ public final class ASTTransformationVisitor extends 
ClassCodeVisitorSupport {
 
             // second pass, call visit on all the collected nodes
             List<Tuple2<ASTTransformation, ASTNode[]>> tuples = new 
ArrayList<>();
-            for (ASTNode[] node : targetNodes) {
-                for (ASTTransformation snt : transforms.get(node[0])) {
-                    tuples.add(new Tuple2<>(snt, node));
+            for (ASTNode[] nodes : targetNodes) {
+                for (ASTTransformation xform : transforms.get(nodes[0])) {
+                    tuples.add(new Tuple2<>(xform, nodes));
                 }
             }
-            tuples.sort(priorityComparator);
+            if (tuples.size() > 1) tuples.sort(priorityComparator);
             for (Tuple2<ASTTransformation, ASTNode[]> tuple : tuples) {
-                if (tuple.getV1() instanceof CompilationUnitAware) {
-                    
((CompilationUnitAware)tuple.getV1()).setCompilationUnit(context.getCompilationUnit());
+                if (tuple.getV1() instanceof CompilationUnitAware unitAware) {
+                    unitAware.setCompilationUnit(context.getCompilationUnit());
                 }
                 tuple.getV1().visit(tuple.getV2(), source);
             }
@@ -237,8 +237,6 @@ public final class ASTTransformationVisitor extends 
ClassCodeVisitorSupport {
                         visitor.source = source;
                         visitor.visitClass(classNode);
                     }, phase.getPhaseNumber());
-                    break;
-
             }
         }
     }

Reply via email to