LGTM w/nit

http://gwt-code-reviews.appspot.com/1460801/diff/1/dev/core/src/com/google/gwt/dev/jjs/impl/GwtAstBuilder.java
File dev/core/src/com/google/gwt/dev/jjs/impl/GwtAstBuilder.java (left):

http://gwt-code-reviews.appspot.com/1460801/diff/1/dev/core/src/com/google/gwt/dev/jjs/impl/GwtAstBuilder.java#oldcode3183
dev/core/src/com/google/gwt/dev/jjs/impl/GwtAstBuilder.java:3183:
s/createSynthic.../createSynthetic...

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

http://gwt-code-reviews.appspot.com/1460801/diff/1/dev/core/src/com/google/gwt/dev/jjs/impl/GwtAstBuilder.java#newcode2020
dev/core/src/com/google/gwt/dev/jjs/impl/GwtAstBuilder.java:2020:
bridgeMethod.freezeParamTypes();
In which way was the original version not being tidy with infos?   It
seems like ultimately, the same info ends up everywhere in the same
place, etc.

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

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

Reply via email to