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