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/04/06 07:07:56 UTC
[2/4] groovy git commit: Trivial refactoring: extract variable
Trivial refactoring: extract variable
(cherry picked from commit ebdfa19)
Project: http://git-wip-us.apache.org/repos/asf/groovy/repo
Commit: http://git-wip-us.apache.org/repos/asf/groovy/commit/09070aa7
Tree: http://git-wip-us.apache.org/repos/asf/groovy/tree/09070aa7
Diff: http://git-wip-us.apache.org/repos/asf/groovy/diff/09070aa7
Branch: refs/heads/GROOVY_2_6_X
Commit: 09070aa7f3ead44b89b92de8dd29e3fd9274259d
Parents: de0cba8
Author: danielsun1106 <re...@hotmail.com>
Authored: Fri Apr 6 14:46:13 2018 +0800
Committer: danielsun1106 <re...@hotmail.com>
Committed: Fri Apr 6 15:07:34 2018 +0800
----------------------------------------------------------------------
.../java/org/codehaus/groovy/control/ResolveVisitor.java | 10 ++++++----
1 file changed, 6 insertions(+), 4 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/groovy/blob/09070aa7/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 6065a6d..bc3630f 100644
--- a/src/main/java/org/codehaus/groovy/control/ResolveVisitor.java
+++ b/src/main/java/org/codehaus/groovy/control/ResolveVisitor.java
@@ -368,15 +368,17 @@ public class ResolveVisitor extends ClassCodeExpressionTransformer {
// test if vanilla name is current class name
if (currentClass == type) return true;
- if (genericParameterNames.get(type.getName()) != null) {
- GenericsType gt = genericParameterNames.get(type.getName());
+ String typeName = type.getName();
+
+ if (genericParameterNames.get(typeName) != null) {
+ GenericsType gt = genericParameterNames.get(typeName);
type.setRedirect(gt.getType());
- type.setGenericsTypes(new GenericsType[]{gt});
+ type.setGenericsTypes(new GenericsType[]{ gt });
type.setGenericsPlaceHolder(true);
return true;
}
- if (currentClass.getNameWithoutPackage().equals(type.getName())) {
+ if (currentClass.getNameWithoutPackage().equals(typeName)) {
type.setRedirect(currentClass);
return true;
}