You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@groovy.apache.org by em...@apache.org on 2022/11/15 15:13:58 UTC

[groovy] 01/02: `CastExpression`: indicate coercion in `getText()` and `toString()`

This is an automated email from the ASF dual-hosted git repository.

emilles pushed a commit to branch GROOVY_3_0_X
in repository https://gitbox.apache.org/repos/asf/groovy.git

commit c59d28f9bd18da13686019aec179e9a68f0ca009
Author: Eric Milles <er...@thomsonreuters.com>
AuthorDate: Tue Nov 15 08:44:51 2022 -0600

    `CastExpression`: indicate coercion in `getText()` and `toString()`
---
 src/main/java/org/codehaus/groovy/ast/expr/CastExpression.java | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/src/main/java/org/codehaus/groovy/ast/expr/CastExpression.java b/src/main/java/org/codehaus/groovy/ast/expr/CastExpression.java
index 4932112b55..303514f02a 100644
--- a/src/main/java/org/codehaus/groovy/ast/expr/CastExpression.java
+++ b/src/main/java/org/codehaus/groovy/ast/expr/CastExpression.java
@@ -82,7 +82,7 @@ public class CastExpression extends Expression {
 
     @Override
     public String toString() {
-        return super.toString() +"[(" + getType().getName() + ") " + expression + "]";
+        return super.toString() + "[" + getText() + "]";
     }
 
     @Override
@@ -102,7 +102,10 @@ public class CastExpression extends Expression {
 
     @Override
     public String getText() {
-        return "(" + getType().toString(false) + ") " + expression.getText(); // TODO: add alternate for "as"?
+        if (isCoerce()) {
+            return expression.getText() + " as " + getType().toString(false);
+        }
+        return "(" + getType().toString(false) + ") " + expression.getText();
     }
 
     @Override