LGTM
http://gwt-code-reviews.appspot.com/1447821/diff/20001/dev/core/src/com/google/gwt/dev/jjs/impl/GenerateJavaAST.java File dev/core/src/com/google/gwt/dev/jjs/impl/GenerateJavaAST.java (right): http://gwt-code-reviews.appspot.com/1447821/diff/20001/dev/core/src/com/google/gwt/dev/jjs/impl/GenerateJavaAST.java#newcode516 dev/core/src/com/google/gwt/dev/jjs/impl/GenerateJavaAST.java:516: } else { Heh, I think I was thinking the else above could be changed to an "else if", to save a level of parens (no biggy, since this class is likely to go away altogether once we are sure the new GwtAstBuilder will stick, which is looking very likely at present) http://gwt-code-reviews.appspot.com/1447821/ -- http://groups.google.com/group/Google-Web-Toolkit-Contributors