Thanks for the review, Bob, and for the helpful iteration, Scott, Ray
C., and John T.

It's committed at r7368.


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
On 2010/01/06 23:02:41, bobv wrote:
Use block quote.

Done.

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
On 2010/01/06 23:02:41, bobv wrote:
Update the comment with *

Done.

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) {
Done.  "filterOutSyntheticMembers".

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:
Done.  Auto-sorting must have produced them.

http://gwt-code-reviews.appspot.com/126817
-- 
http://groups.google.com/group/Google-Web-Toolkit-Contributors

Reply via email to