LGTM, just some nits
http://gwt-code-reviews.appspot.com/126817/diff/2024/2038 File dev/core/src/com/google/gwt/dev/jjs/impl/JsniRefLookup.java (right): http://gwt-code-reviews.appspot.com/126817/diff/2024/2038#newcode118 Line 118: // Backward compatibility: allow accessing bridge methods with full Use block quote. http://gwt-code-reviews.appspot.com/126817/diff/2024/2040 File dev/core/src/com/google/gwt/dev/js/rhino/TokenStream.java (right): http://gwt-code-reviews.appspot.com/126817/diff/2024/2040#newcode1377 Line 1377: // First check for the special case of ?? as the parameter list, indicating Update the comment with * http://gwt-code-reviews.appspot.com/126817/diff/2024/2039 File dev/core/src/com/google/gwt/dev/shell/DispatchClassInfo.java (right): http://gwt-code-reviews.appspot.com/126817/diff/2024/2039#newcode236 Line 236: private List<Member> removeSyntheticMembers(Collection<Member> members) { Since this doesn't mutate the input collection, rename it to "filter" http://gwt-code-reviews.appspot.com/126817/diff/2024/2041 File dev/core/src/com/google/gwt/dev/util/JsniRef.java (right): http://gwt-code-reviews.appspot.com/126817/diff/2024/2041#newcode128 Line 128: Unnecessary blank lines. http://gwt-code-reviews.appspot.com/126817/diff/2024/2026 File dev/core/test/com/google/gwt/dev/jjs/impl/JsniRefLookupTest.java (right): http://gwt-code-reviews.appspot.com/126817/diff/2024/2026#newcode50 Line 50: public void testConstructors() { Sort order. http://gwt-code-reviews.appspot.com/126817
-- http://groups.google.com/group/Google-Web-Toolkit-Contributors
