commit add9eeab660a2be6d19c1d1b5806b03f0ed1d61b
author scottb@google.com <scottb@google.com@8db76d5a-ed1c-0410-87a9-c151d255dfc7> Mon Jun 13 19:05:05 2011 +0000
committer scottb@google.com <scottb@google.com@8db76d5a-ed1c-0410-87a9-c151d255dfc7> Mon Jun 13 19:05:05 2011 +0000
tree 173a5beb82fd09277dd2c8df2373f936d5eeadb5
parent 4378b399bb04562a2fa79020900d0e9cc6bec914

Fix enum switch construction in GwtAstBuilder.

The existing code breaks on switch(Integer), it tries to tack an ordinal() call on the end of the unbox.

http://gwt-code-reviews.appspot.com/1450815/

Review by: zundel@google.com

git-svn-id: https://google-web-toolkit.googlecode.com/svn/trunk@10328 8db76d5a-ed1c-0410-87a9-c151d255dfc7

dev/core/src/com/google/gwt/dev/jjs/impl/GwtAstBuilder.java

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 50d98cb..120e4ed 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
@@ -1366,8 +1366,8 @@
         JBlock block = popBlock(info, x.statements);
         JExpression expression = pop(x.expression);
 
-        if (x.expression.resolvedType instanceof ReferenceBinding) {
-          // Must be an enum; synthesize a call to ordinal().
+        if (x.expression.resolvedType.isEnum()) {
+          // synthesize a call to ordinal().
           ReferenceBinding javaLangEnum = scope.getJavaLangEnum();
           MethodBinding ordinal = javaLangEnum.getMethods(ORDINAL)[0];
           expression = new JMethodCall(info, expression, typeMap.get(ordinal));