You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@groovy.apache.org by su...@apache.org on 2018/07/13 06:43:31 UTC

groovy git commit: Trivial refactoring: remove useless code

Repository: groovy
Updated Branches:
  refs/heads/GROOVY_2_5_X ecd98b52c -> 6a72293e3


Trivial refactoring: remove useless code

(cherry picked from commit 61fa507)


Project: http://git-wip-us.apache.org/repos/asf/groovy/repo
Commit: http://git-wip-us.apache.org/repos/asf/groovy/commit/6a72293e
Tree: http://git-wip-us.apache.org/repos/asf/groovy/tree/6a72293e
Diff: http://git-wip-us.apache.org/repos/asf/groovy/diff/6a72293e

Branch: refs/heads/GROOVY_2_5_X
Commit: 6a72293e3974a0cbdd9a7a71e97081f9d0bc7918
Parents: ecd98b5
Author: sunlan <su...@apache.org>
Authored: Fri Jul 13 14:42:47 2018 +0800
Committer: sunlan <su...@apache.org>
Committed: Fri Jul 13 14:43:15 2018 +0800

----------------------------------------------------------------------
 src/main/java/org/codehaus/groovy/control/ResolveVisitor.java | 1 -
 1 file changed, 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/groovy/blob/6a72293e/src/main/java/org/codehaus/groovy/control/ResolveVisitor.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/codehaus/groovy/control/ResolveVisitor.java b/src/main/java/org/codehaus/groovy/control/ResolveVisitor.java
index 7853492..65ea9cf 100644
--- a/src/main/java/org/codehaus/groovy/control/ResolveVisitor.java
+++ b/src/main/java/org/codehaus/groovy/control/ResolveVisitor.java
@@ -1019,7 +1019,6 @@ public class ResolveVisitor extends ClassCodeExpressionTransformer {
                 // referencedClassVariables, but must be removed
                 // for each parentscope too
                 for (VariableScope scope = currentScope; scope != null && !scope.isRoot(); scope = scope.getParent()) {
-                    if (scope.isRoot()) break;
                     if (scope.removeReferencedClassVariable(ve.getName()) == null) break;
                 }
                 ClassExpression ce = new ClassExpression(t);