LGTM. Thanks for the patch, the code is much cleaner now.
http://gwt-code-reviews.appspot.com/250801/diff/2001/18 File bikeshed/src/com/google/gwt/requestfactory/shared/RequestFactory.java (right): http://gwt-code-reviews.appspot.com/250801/diff/2001/18#newcode3 bikeshed/src/com/google/gwt/requestfactory/shared/RequestFactory.java:3: * unnecessary change (in other lines as well). http://gwt-code-reviews.appspot.com/250801/diff/2001/18#newcode33 bikeshed/src/com/google/gwt/requestfactory/shared/RequestFactory.java:33: is this method necessary? Seems like a convenience method that can be removed to keep the api surface area low. http://gwt-code-reviews.appspot.com/250801/show -- http://groups.google.com/group/Google-Web-Toolkit-Contributors To unsubscribe from this group, send email to google-web-toolkit-contributors+unsubscribegooglegroups.com or reply to this email with the words "REMOVE ME" as the subject.
