LGTM, but could we handle if (false) also? It can only make our optimizations better.
http://gwt-code-reviews.appspot.com/34835/diff/1/3 File dev/core/src/com/google/gwt/dev/jjs/ast/JIfStatement.java (right): http://gwt-code-reviews.appspot.com/34835/diff/1/3#newcode64 Line 64: if (thenStmt != null && thenStmt.unconditionalControlBreak()) { Can we handle all three cases? 1) condition true + unconditional then 2) condition false + unconditional false 3) both branches unconditional http://gwt-code-reviews.appspot.com/34835 --~--~---------~--~----~------------~-------~--~----~ http://groups.google.com/group/Google-Web-Toolkit-Contributors -~----------~----~----~----~------~----~------~--~---
