http://gwt-code-reviews.appspot.com/159803/diff/1/3
File dev/core/src/com/google/gwt/dev/jjs/ast/JConstructor.java (right):

http://gwt-code-reviews.appspot.com/159803/diff/1/3#newcode80
Line 80: if (expr instanceof JMethodCall) {
I see.  It looks good now.

http://gwt-code-reviews.appspot.com/159803/diff/1/25
File dev/core/src/com/google/gwt/dev/jjs/impl/RemoveEmptySuperCalls.java
(right):

http://gwt-code-reviews.appspot.com/159803/diff/1/25#newcode50
Line 50: ctx.replaceMe(multi.makeStatement());
That dependency will surely come right back.  For example, it's an
oversight that Simplifier.cast does not check whether the cast is
trivial.  More broadly, Simplifier should be able to construct any kind
of expression or statement node.  This very patch includes a new node
type that can only be created via a JProgram object.

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

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

Reply via email to