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 14:55:47 UTC

[groovy] branch GROOVY_4_0_X updated: `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_4_0_X
in repository https://gitbox.apache.org/repos/asf/groovy.git


The following commit(s) were added to refs/heads/GROOVY_4_0_X by this push:
     new df48b4b4de `CastExpression`: indicate coercion in `getText()` and `toString()`
df48b4b4de is described below

commit df48b4b4dead0e9cd490b9784848bb162377ded0
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 b760573d9d..7c35eda473 100644
--- a/src/main/java/org/codehaus/groovy/ast/expr/CastExpression.java
+++ b/src/main/java/org/codehaus/groovy/ast/expr/CastExpression.java
@@ -88,7 +88,7 @@ public class CastExpression extends Expression {
 
     @Override
     public String toString() {
-        return super.toString() +"[(" + getType().getName() + ") " + expression + "]";
+        return super.toString() + "[" + getText() + "]";
     }
 
     @Override
@@ -108,7 +108,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