LGTM w/ nits.

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

http://gwt-code-reviews.appspot.com/691801/diff/1/2#newcode497
dev/core/src/com/google/gwt/dev/jjs/impl/BuildTypeMap.java:497:
typeMap.get(superInterfaceBinding);
This one actually formats on the same line for me.

http://gwt-code-reviews.appspot.com/691801/diff/1/2#newcode737
dev/core/src/com/google/gwt/dev/jjs/impl/BuildTypeMap.java:737:
@SuppressWarnings({"SuspiciousMethodCalls"})
What's this?

http://gwt-code-reviews.appspot.com/691801/show

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

Reply via email to