You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@groovy.apache.org by pa...@apache.org on 2021/09/26 12:59:58 UTC
[groovy] branch GROOVY_3_0_X updated: GROOVY-10262:
MethodNode#getText can output incorrect modifiers
This is an automated email from the ASF dual-hosted git repository.
paulk pushed a commit to branch GROOVY_3_0_X
in repository https://gitbox.apache.org/repos/asf/groovy.git
The following commit(s) were added to refs/heads/GROOVY_3_0_X by this push:
new 38e8fe5 GROOVY-10262: MethodNode#getText can output incorrect modifiers
38e8fe5 is described below
commit 38e8fe556b0d1251b143298618d158e15cb31987
Author: Paul King <pa...@asert.com.au>
AuthorDate: Sun Sep 26 22:58:45 2021 +1000
GROOVY-10262: MethodNode#getText can output incorrect modifiers
---
src/main/java/org/codehaus/groovy/ast/MethodNode.java | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/src/main/java/org/codehaus/groovy/ast/MethodNode.java b/src/main/java/org/codehaus/groovy/ast/MethodNode.java
index f91cd7a..f2214e4 100644
--- a/src/main/java/org/codehaus/groovy/ast/MethodNode.java
+++ b/src/main/java/org/codehaus/groovy/ast/MethodNode.java
@@ -23,6 +23,7 @@ import org.codehaus.groovy.ast.stmt.BlockStatement;
import org.codehaus.groovy.ast.stmt.Statement;
import org.objectweb.asm.Opcodes;
+import java.lang.reflect.Modifier;
import java.util.List;
import java.util.Optional;
@@ -280,7 +281,8 @@ public class MethodNode extends AnnotatedNode implements Opcodes {
String retType = AstToTextHelper.getClassText(returnType);
String exceptionTypes = AstToTextHelper.getThrowsClauseText(exceptions);
String params = AstToTextHelper.getParametersText(parameters);
- return AstToTextHelper.getModifiersText(modifiers) + " " + retType + " " + name + "(" + params + ") " + exceptionTypes + " { ... }";
+ int mask = this instanceof ConstructorNode ? Modifier.constructorModifiers() : Modifier.methodModifiers();
+ return AstToTextHelper.getModifiersText(modifiers & mask) + " " + retType + " " + name + "(" + params + ") " + exceptionTypes + " { ... }";
}
@Override