Misc gwtc cleanups.
http://gwt-code-reviews.appspot.com/1446802/
Review by: zundel@google.com
git-svn-id: https://google-web-toolkit.googlecode.com/svn/trunk@10179 8db76d5a-ed1c-0410-87a9-c151d255dfc7
diff --git a/dev/core/src/com/google/gwt/core/ext/linker/impl/StandardSymbolData.java b/dev/core/src/com/google/gwt/core/ext/linker/impl/StandardSymbolData.java
index 5112b94..fc57185 100644
--- a/dev/core/src/com/google/gwt/core/ext/linker/impl/StandardSymbolData.java
+++ b/dev/core/src/com/google/gwt/core/ext/linker/impl/StandardSymbolData.java
@@ -137,7 +137,7 @@
@Override
public String toString() {
- return methodSig != null ? methodSig : className;
+ return isClass() ? className : getJsniIdent();
}
private void readObject(ObjectInputStream in) throws IOException,
diff --git a/dev/core/src/com/google/gwt/dev/jjs/ast/JNewArray.java b/dev/core/src/com/google/gwt/dev/jjs/ast/JNewArray.java
index 4ceec03..21e2f47 100644
--- a/dev/core/src/com/google/gwt/dev/jjs/ast/JNewArray.java
+++ b/dev/core/src/com/google/gwt/dev/jjs/ast/JNewArray.java
@@ -16,8 +16,8 @@
package com.google.gwt.dev.jjs.ast;
import com.google.gwt.dev.jjs.SourceInfo;
+import com.google.gwt.dev.util.collect.Lists;
-import java.util.ArrayList;
import java.util.List;
/**
@@ -26,8 +26,6 @@
public class JNewArray extends JExpression {
public static JNewArray createDims(SourceInfo info, JArrayType arrayType, List<JExpression> dims) {
- List<JClassLiteral> classLiterals = new ArrayList<JClassLiteral>();
-
// Produce all class literals that will eventually get generated.
int realDims = 0;
for (JExpression dim : dims) {
@@ -37,11 +35,12 @@
++realDims;
}
+ List<JClassLiteral> classLiterals = Lists.create();
JType cur = arrayType;
for (int i = 0; i < realDims; ++i) {
// Walk down each type from most dims to least.
JClassLiteral classLit = new JClassLiteral(info.makeChild(), cur);
- classLiterals.add(classLit);
+ classLiterals = Lists.add(classLiterals, classLit);
cur = ((JArrayType) cur).getElementType();
}
return new JNewArray(info, arrayType, dims, null, classLiterals);
@@ -49,8 +48,8 @@
public static JNewArray createInitializers(SourceInfo info, JArrayType arrayType,
List<JExpression> initializers) {
- List<JClassLiteral> classLiterals = new ArrayList<JClassLiteral>();
- classLiterals.add(new JClassLiteral(info.makeChild(), arrayType));
+ List<JClassLiteral> classLiterals =
+ Lists.create(new JClassLiteral(info.makeChild(), arrayType));
return new JNewArray(info, arrayType, null, initializers, classLiterals);
}
@@ -75,7 +74,7 @@
}
public JArrayType getArrayType() {
- return (JArrayType) type.getUnderlyingType();
+ return type;
}
/**
diff --git a/dev/core/src/com/google/gwt/dev/jjs/impl/GwtAstBuilder.java b/dev/core/src/com/google/gwt/dev/jjs/impl/GwtAstBuilder.java
index aa7b0a4..c58dfa3 100644
--- a/dev/core/src/com/google/gwt/dev/jjs/impl/GwtAstBuilder.java
+++ b/dev/core/src/com/google/gwt/dev/jjs/impl/GwtAstBuilder.java
@@ -653,7 +653,7 @@
*/
if (!hasExplicitThis) {
ReferenceBinding declaringClass = (ReferenceBinding) x.binding.declaringClass.erasure();
- if (declaringClass instanceof NestedTypeBinding) {
+ if (isNested(declaringClass)) {
NestedTypeBinding nestedBinding = (NestedTypeBinding) declaringClass;
if (nestedBinding.enclosingInstances != null) {
for (SyntheticArgumentBinding arg : nestedBinding.enclosingInstances) {
@@ -768,7 +768,7 @@
if (x.isSuperAccess()) {
JExpression qualifier = pop(x.qualification);
ReferenceBinding superClass = x.binding.declaringClass;
- boolean nestedSuper = superClass.isNestedType() && !superClass.isStatic();
+ boolean nestedSuper = isNested(superClass);
if (nestedSuper) {
processSuperCallThisArgs(superClass, call, qualifier, x.qualification);
}
@@ -779,7 +779,7 @@
} else {
assert (x.qualification == null);
ReferenceBinding declaringClass = x.binding.declaringClass;
- boolean nested = declaringClass.isNestedType() && !declaringClass.isStatic();
+ boolean nested = isNested(declaringClass);
if (nested) {
processThisCallThisArgs(declaringClass, call);
}
@@ -1553,7 +1553,8 @@
// Map synthetic arguments for outer this.
ReferenceBinding declaringClass = (ReferenceBinding) x.binding.declaringClass.erasure();
- if (declaringClass.isNestedType() && !declaringClass.isStatic()) {
+ boolean isNested = isNested(declaringClass);
+ if (isNested) {
NestedTypeBinding nestedBinding = (NestedTypeBinding) declaringClass;
if (nestedBinding.enclosingInstances != null) {
for (int i = 0; i < nestedBinding.enclosingInstances.length; ++i) {
@@ -1571,7 +1572,7 @@
}
// Map synthetic arguments for locals.
- if (declaringClass.isNestedType() && !declaringClass.isStatic()) {
+ if (isNested) {
// add synthetic args for locals
NestedTypeBinding nestedBinding = (NestedTypeBinding) declaringClass;
// add synthetic args for outer this and locals
@@ -1886,7 +1887,7 @@
*/
int index = 0;
SourceTypeBinding binding = x.binding;
- if (binding.isNestedType() && !binding.isStatic()) {
+ if (isNested(binding)) {
// add synthetic fields for outer this and locals
assert (type instanceof JClassType);
NestedTypeBinding nestedBinding = (NestedTypeBinding) binding;
@@ -2487,24 +2488,20 @@
// Synthetic args for inner classes
ReferenceBinding targetBinding = (ReferenceBinding) b.declaringClass.erasure();
- NestedTypeBinding nestedBinding = null;
- if (targetBinding.isNestedType() && !targetBinding.isStatic()) {
- nestedBinding = (NestedTypeBinding) targetBinding;
- }
- if (nestedBinding != null) {
+ boolean isNested = isNested(targetBinding);
+ if (isNested) {
// Synthetic this args for inner classes
- if (nestedBinding.enclosingInstances != null) {
+ if (targetBinding.syntheticEnclosingInstanceTypes() != null) {
ReferenceBinding checkedTargetType =
targetBinding.isAnonymousType() ? (ReferenceBinding) targetBinding.superclass()
.erasure() : targetBinding;
ReferenceBinding targetEnclosingType = checkedTargetType.enclosingType();
- for (SyntheticArgumentBinding arg : nestedBinding.enclosingInstances) {
- TypeBinding argType = arg.type.erasure();
+ for (ReferenceBinding argType : targetBinding.syntheticEnclosingInstanceTypes()) {
+ argType = (ReferenceBinding) argType.erasure();
if (qualifier != null && argType == targetEnclosingType) {
call.addArg(qualExpr);
} else {
- JExpression thisRef =
- makeThisReference(info, (ReferenceBinding) argType, false, scope);
+ JExpression thisRef = makeThisReference(info, argType, false, scope);
call.addArg(thisRef);
}
}
@@ -2515,10 +2512,10 @@
call.addArgs(arguments);
// Synthetic args for inner classes
- if (nestedBinding != null) {
+ if (isNested) {
// Synthetic locals for local classes
- if (nestedBinding.outerLocalVariables != null) {
- for (SyntheticArgumentBinding arg : nestedBinding.outerLocalVariables) {
+ if (targetBinding.syntheticOuterLocalVariables() != null) {
+ for (SyntheticArgumentBinding arg : targetBinding.syntheticOuterLocalVariables()) {
LocalVariableBinding targetVariable = arg.actualOuterLocalVariable;
VariableBinding[] path = scope.getEmulationPath(targetVariable);
assert path.length == 1;
@@ -2813,6 +2810,10 @@
return stringInterner.intern(s);
}
+ static boolean isNested(ReferenceBinding binding) {
+ return binding.isNestedType() && !binding.isStatic();
+ }
+
/**
* Returns <code>true</code> if JDT optimized the condition to
* <code>false</code>.
@@ -3065,6 +3066,7 @@
JDeclaredType enclosingType = (JDeclaredType) typeMap.get(declaringClass);
assert !enclosingType.isExternal();
JMethod method;
+ boolean isNested = isNested(declaringClass);
if (x.isConstructor()) {
method = new JConstructor(info, (JClassType) enclosingType);
if (x.binding.declaringClass.isEnum()) {
@@ -3075,7 +3077,7 @@
method));
}
// add synthetic args for outer this
- if (declaringClass.isNestedType() && !declaringClass.isStatic()) {
+ if (isNested) {
NestedTypeBinding nestedBinding = (NestedTypeBinding) declaringClass;
if (nestedBinding.enclosingInstances != null) {
for (int i = 0; i < nestedBinding.enclosingInstances.length; ++i) {
@@ -3099,7 +3101,7 @@
createParameters(method, x);
if (x.isConstructor()) {
- if (declaringClass.isNestedType() && !declaringClass.isStatic()) {
+ if (isNested) {
// add synthetic args for locals
NestedTypeBinding nestedBinding = (NestedTypeBinding) declaringClass;
// add synthetic args for outer this and locals
diff --git a/dev/core/src/com/google/gwt/dev/jjs/impl/ReplaceRebinds.java b/dev/core/src/com/google/gwt/dev/jjs/impl/ReplaceRebinds.java
index 60e4649..94056ee 100644
--- a/dev/core/src/com/google/gwt/dev/jjs/impl/ReplaceRebinds.java
+++ b/dev/core/src/com/google/gwt/dev/jjs/impl/ReplaceRebinds.java
@@ -100,45 +100,35 @@
JStringLiteral stringLiteral = (JStringLiteral) arg0;
String stringValue = stringLiteral.getValue();
- HasName named = null;
+ JNode node = null;
JDeclaredType refType;
JsniRef ref = JsniRef.parse(stringValue);
if (ref != null) {
final List<String> errors = new ArrayList<String>();
- JNode node =
- JsniRefLookup.findJsniRefTarget(ref, program, new JsniRefLookup.ErrorReporter() {
- public void reportError(String error) {
- errors.add(error);
- }
- });
-
+ node = JsniRefLookup.findJsniRefTarget(ref, program, new JsniRefLookup.ErrorReporter() {
+ public void reportError(String error) {
+ errors.add(error);
+ }
+ });
if (!errors.isEmpty()) {
for (String error : errors) {
logger.log(TreeLogger.ERROR, error);
}
}
-
- if (node instanceof HasName) {
- named = (HasName) node;
- }
-
} else {
// See if it's just @foo.Bar, which would result in the class seed
- refType =
+ node =
program.getFromTypeMap(stringValue.charAt(0) == '@' ? stringValue.substring(1)
: stringValue);
- if (refType != null) {
- named = refType;
- }
}
-
- if (named == null) {
+ if (node == null) {
// Not found, must be null
ctx.replaceMe(JNullLiteral.INSTANCE);
} else {
- ctx.replaceMe(new JNameOf(x.getSourceInfo(), program.getTypeJavaLangString(), named));
+ ctx.replaceMe(new JNameOf(x.getSourceInfo(), program.getTypeJavaLangString(),
+ (HasName) node));
}
}
}
