LGTM w/ nits.

http://gwt-code-reviews.appspot.com/1384806/diff/2001/user/test/com/google/gwt/requestfactory/shared/ServiceInheritanceTest.java
File
user/test/com/google/gwt/requestfactory/shared/ServiceInheritanceTest.java
(right):

http://gwt-code-reviews.appspot.com/1384806/diff/2001/user/test/com/google/gwt/requestfactory/shared/ServiceInheritanceTest.java#newcode96
user/test/com/google/gwt/requestfactory/shared/ServiceInheritanceTest.java:96:
// implementations in the tests
Use a block comment for multi-line comments to allow eclipse to re-flow
the text.

http://gwt-code-reviews.appspot.com/1384806/diff/2001/user/test/com/google/gwt/requestfactory/shared/ServiceInheritanceTest.java#newcode132
user/test/com/google/gwt/requestfactory/shared/ServiceInheritanceTest.java:132:
public void testInvokeInheritedMethod() {
Sort methods.

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

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

Reply via email to