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 2025/01/18 10:19:17 UTC
(groovy) branch master updated: Trivial refactoring: extract constant
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 757cc1b710 Trivial refactoring: extract constant
757cc1b710 is described below
commit 757cc1b7104c46c2243ba1415440dad8be9c5b3c
Author: Daniel Sun <su...@apache.org>
AuthorDate: Sat Jan 18 19:19:06 2025 +0900
Trivial refactoring: extract constant
---
.../codehaus/groovy/classgen/asm/sc/StaticTypesLambdaWriter.java | 9 +++++----
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/src/main/java/org/codehaus/groovy/classgen/asm/sc/StaticTypesLambdaWriter.java b/src/main/java/org/codehaus/groovy/classgen/asm/sc/StaticTypesLambdaWriter.java
index 2eda78ee74..b804702754 100644
--- a/src/main/java/org/codehaus/groovy/classgen/asm/sc/StaticTypesLambdaWriter.java
+++ b/src/main/java/org/codehaus/groovy/classgen/asm/sc/StaticTypesLambdaWriter.java
@@ -85,6 +85,7 @@ public class StaticTypesLambdaWriter extends LambdaWriter implements AbstractFun
private static final String IS_GENERATED_CONSTRUCTOR = "__IS_GENERATED_CONSTRUCTOR";
private static final String LAMBDA_SHARED_VARIABLES = "__LAMBDA_SHARED_VARIABLES";
+ private static final String DO_CALL = "doCall";
private final Map<Expression, ClassNode> lambdaClassNodes = new HashMap<>();
private final StaticTypesClosureWriter staticTypesClosureWriter;
@@ -110,7 +111,7 @@ public class StaticTypesLambdaWriter extends LambdaWriter implements AbstractFun
}
ClassNode lambdaClass = getOrAddLambdaClass(expression, abstractMethod);
- MethodNode lambdaMethod = lambdaClass.getMethods("doCall").get(0);
+ MethodNode lambdaMethod = lambdaClass.getMethods(DO_CALL).get(0);
boolean canDeserialize = controller.getClassNode().hasMethod(createDeserializeLambdaMethodName(lambdaClass), createDeserializeLambdaMethodParams());
if (!canDeserialize) {
@@ -214,8 +215,8 @@ public class StaticTypesLambdaWriter extends LambdaWriter implements AbstractFun
}
private ClassNode getOrAddLambdaClass(final LambdaExpression expression, final MethodNode abstractMethod) {
- return lambdaClassNodes.computeIfAbsent(expression, key -> {
- ClassNode lambdaClass = createLambdaClass(expression, ACC_FINAL | ACC_PUBLIC | ACC_STATIC, abstractMethod);
+ return lambdaClassNodes.computeIfAbsent(expression, expr -> {
+ ClassNode lambdaClass = createLambdaClass((LambdaExpression) expr, ACC_FINAL | ACC_PUBLIC | ACC_STATIC, abstractMethod);
controller.getAcg().addInnerClass(lambdaClass);
lambdaClass.addInterface(GENERATED_LAMBDA_TYPE);
lambdaClass.putNodeMetaData(StaticCompilationMetadataKeys.STATIC_COMPILE_NODE, Boolean.TRUE);
@@ -276,7 +277,7 @@ public class StaticTypesLambdaWriter extends LambdaWriter implements AbstractFun
expression.putNodeMetaData(LAMBDA_SHARED_VARIABLES, localVariableParameters);
MethodNode doCallMethod = lambdaClass.addMethod(
- "doCall",
+ DO_CALL,
ACC_PUBLIC,
abstractMethod.getReturnType(),
parametersWithExactType.clone(),