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 2023/09/11 00:27:43 UTC
[groovy] 01/03: Improve method selection for script body in SecureASTCustomizer
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
commit 5e8e09f4a69db6881a1ea3a086f8069b105a002a
Author: Jochen Theodorou <bl...@gmx.org>
AuthorDate: Tue Aug 29 18:03:40 2023 +0200
Improve method selection for script body in SecureASTCustomizer
---
.../org/codehaus/groovy/control/customizers/SecureASTCustomizer.java | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/main/java/org/codehaus/groovy/control/customizers/SecureASTCustomizer.java b/src/main/java/org/codehaus/groovy/control/customizers/SecureASTCustomizer.java
index 74594d2743..c337a92c8a 100644
--- a/src/main/java/org/codehaus/groovy/control/customizers/SecureASTCustomizer.java
+++ b/src/main/java/org/codehaus/groovy/control/customizers/SecureASTCustomizer.java
@@ -924,7 +924,7 @@ public class SecureASTCustomizer extends CompilationCustomizer {
List<MethodNode> methods = owner.getMethods();
for (MethodNode method : methods) {
if (method.getDeclaringClass() == owner && !method.isSynthetic()) {
- if ("main".equals(method.getName()) || "run".equals(method.getName()) && owner.isScriptBody()) continue;
+ if (("main".equals(method.getName()) || "run".equals(method.getName())) && owner.isScriptBody()) continue;
result.add(method);
}
}