Reviewers: jbrosenberg,

Description:
An empty for statement can generate an ICE.

for (Integer xyzzy : new ArrayList<Integer>())
  ; // TODO


Please review this at http://gwt-code-reviews.appspot.com/1400801/

Affected files:
  M dev/core/src/com/google/gwt/dev/jjs/impl/GenerateJavaAST.java


Index: dev/core/src/com/google/gwt/dev/jjs/impl/GenerateJavaAST.java
diff --git a/dev/core/src/com/google/gwt/dev/jjs/impl/GenerateJavaAST.java b/dev/core/src/com/google/gwt/dev/jjs/impl/GenerateJavaAST.java index b0f4fbb834ed27a6146ebb7977b2ef1654bb58e2..91ce358a8ebdc04a3022d97e636a650bc4405e07 100644
--- a/dev/core/src/com/google/gwt/dev/jjs/impl/GenerateJavaAST.java
+++ b/dev/core/src/com/google/gwt/dev/jjs/impl/GenerateJavaAST.java
@@ -1639,7 +1639,9 @@ public class GenerateJavaAST {
         body = (JBlock) action;
       } else {
         body = new JBlock(info);
-        body.addStmt(action);
+        if (action != null) {
+          body.addStmt(action);
+        }
       }

       JLocal elementVar = (JLocal) typeMap.get(x.elementVariable.binding);


--
http://groups.google.com/group/Google-Web-Toolkit-Contributors

Reply via email to