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 2020/07/03 08:53:44 UTC

[groovy] branch GROOVY_3_0_X updated: GROOVY-9607 Class generation regression: (revert one change from GROOVY-9344)

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

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


The following commit(s) were added to refs/heads/GROOVY_3_0_X by this push:
     new f95deec  GROOVY-9607 Class generation regression: (revert one change from GROOVY-9344)
f95deec is described below

commit f95deec3531e9f751e24e4d632823de62fed7a36
Author: Paul King <pa...@asert.com.au>
AuthorDate: Fri Jul 3 18:53:36 2020 +1000

    GROOVY-9607 Class generation regression: (revert one change from GROOVY-9344)
---
 .../org/codehaus/groovy/classgen/asm/sc/StaticTypesTypeChooser.java     | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/main/java/org/codehaus/groovy/classgen/asm/sc/StaticTypesTypeChooser.java b/src/main/java/org/codehaus/groovy/classgen/asm/sc/StaticTypesTypeChooser.java
index f641e50..534959c 100644
--- a/src/main/java/org/codehaus/groovy/classgen/asm/sc/StaticTypesTypeChooser.java
+++ b/src/main/java/org/codehaus/groovy/classgen/asm/sc/StaticTypesTypeChooser.java
@@ -34,7 +34,7 @@ import org.codehaus.groovy.transform.stc.StaticTypesMarker;
 public class StaticTypesTypeChooser extends StatementMetaTypeChooser {
     @Override
     public ClassNode resolveType(final Expression exp, final ClassNode current) {
-        Expression target = exp instanceof VariableExpression && !((VariableExpression) exp).isClosureSharedVariable() ? getTarget((VariableExpression) exp) : exp;
+        Expression target = exp instanceof VariableExpression ? getTarget((VariableExpression) exp) : exp;
 
         ClassNode inferredType = target.getNodeMetaData(StaticTypesMarker.DECLARATION_INFERRED_TYPE);
         if (inferredType == null) {