blackdrag commented on code in PR #2500:
URL: https://github.com/apache/groovy/pull/2500#discussion_r3159694868


##########
src/main/java/org/codehaus/groovy/transform/ASTTransformationVisitor.java:
##########
@@ -93,6 +93,22 @@ private ASTTransformationVisitor(final CompilePhase phase, 
final ASTTransformati
         this.context = context;
     }
 
+    /**
+     * Returns a primary class node operation that invokes any AST 
transformations
+     * registered for {@code phase} on each class. Intended for callers that 
need
+     * to drive transform invocation outside the standard phase wiring (for 
example,
+     * {@code JavaAwareCompilationUnit} runs CONVERSION-phase transforms 
before the
+     * stub generator emits stubs — see GEP-21).
+     */
+    public static 
org.codehaus.groovy.control.CompilationUnit.IPrimaryClassNodeOperation
+            invocationOperation(final CompilePhase phase, final 
ASTTransformationsContext context) {

Review Comment:
   concept wise I am not so happy with using IPrimaryClassNodeOperation here. 
It is more of an internal interface of CompilationUnit, it should not be used 
directly outside really.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to