[groovy] branch master updated: set source position

[pa...@apache.org]

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


The following commit(s) were added to refs/heads/master by this push:
     new 756676e  set source position
756676e is described below

commit 756676e71083d420ef9df8b1c9c3bf3f20ba6e6b
Author: Paul King <pa...@asert.com.au>
AuthorDate: Sat May 25 16:42:22 2019 +1000

    set source position
---
 .../transform/sc/transformers/VariableExpressionTransformer.java    | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/main/java/org/codehaus/groovy/transform/sc/transformers/VariableExpressionTransformer.java b/src/main/java/org/codehaus/groovy/transform/sc/transformers/VariableExpressionTransformer.java
index b908425..6e490c1 100644
--- a/src/main/java/org/codehaus/groovy/transform/sc/transformers/VariableExpressionTransformer.java
+++ b/src/main/java/org/codehaus/groovy/transform/sc/transformers/VariableExpressionTransformer.java
@@ -46,13 +46,13 @@ public class VariableExpressionTransformer {
 
     private static Expression tryTransformDelegateToProperty(VariableExpression expr) {
         // we need to transform variable expressions that go to a delegate
-        // to a property expression, as ACG would loose the information
-        // in processClassVariable before it reaches any makeCall, that could
-        // handle it
+        // to a property expression, as ACG would lose the information in
+        // processClassVariable before it reaches any makeCall, that could handle it
         Object val = expr.getNodeMetaData(StaticTypesMarker.IMPLICIT_RECEIVER);
         if (val == null) return null;
         // TODO handle the owner and delegate cases better for nested scenarios and potentially remove the need for the implicit this case
         VariableExpression receiver = new VariableExpression("owner".equals(val) ? (String) val : "delegate".equals(val) ? (String) val : "this");
+        receiver.setSourcePosition(expr);
         PropertyExpression pexp = new PropertyExpression(receiver, expr.getName());
         pexp.copyNodeMetaData(expr);
         pexp.setImplicitThis(true);