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 2019/12/02 01:41:32 UTC
[groovy] 03/03: GROOVY-9183: refactor to simplify (closes #1109)
This is an automated email from the ASF dual-hosted git repository.
paulk pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/groovy.git
commit db5fc65ca489685ba246d14d80bc1132cd566ba5
Author: Paul King <pa...@asert.com.au>
AuthorDate: Mon Dec 2 11:41:16 2019 +1000
GROOVY-9183: refactor to simplify (closes #1109)
---
.../codehaus/groovy/transform/NamedVariantASTTransformation.java | 9 ++-------
1 file changed, 2 insertions(+), 7 deletions(-)
diff --git a/src/main/java/org/codehaus/groovy/transform/NamedVariantASTTransformation.java b/src/main/java/org/codehaus/groovy/transform/NamedVariantASTTransformation.java
index 9316a07..146ad7f 100644
--- a/src/main/java/org/codehaus/groovy/transform/NamedVariantASTTransformation.java
+++ b/src/main/java/org/codehaus/groovy/transform/NamedVariantASTTransformation.java
@@ -49,6 +49,7 @@ import static org.apache.groovy.ast.tools.ClassNodeUtils.addGeneratedConstructor
import static org.apache.groovy.ast.tools.ClassNodeUtils.addGeneratedMethod;
import static org.apache.groovy.ast.tools.ClassNodeUtils.isInnerClass;
import static org.apache.groovy.ast.tools.VisibilityUtils.getVisibility;
+import static org.codehaus.groovy.antlr.PrimitiveHelper.getDefaultValueForPrimitive;
import static org.codehaus.groovy.ast.ClassHelper.MAP_TYPE;
import static org.codehaus.groovy.ast.ClassHelper.STRING_TYPE;
import static org.codehaus.groovy.ast.ClassHelper.boolean_TYPE;
@@ -205,13 +206,7 @@ public class NamedVariantASTTransformation extends AbstractASTTransformation {
}
private Expression getDefaultExpression(ClassNode pType) {
- if (isPrimitiveType(pType)) {
- if (boolean_TYPE.equals(pType)) {
- return constX(Boolean.FALSE);
- }
- return castX(getWrapper(pType), constX(0));
- }
- return nullX();
+ return Optional.ofNullable(getDefaultValueForPrimitive(pType)).orElse(nullX());
}
private boolean hasDuplicates(final MethodNode mNode, final List<String> propNames, final String next) {