LGTM. Only cosmetic nits

http://gwt-code-reviews.appspot.com/1130801/diff/21001/4006
File user/src/com/google/gwt/autobean/shared/AutoBeanFactory.java
(right):

http://gwt-code-reviews.appspot.com/1130801/diff/21001/4006#newcode85
user/src/com/google/gwt/autobean/shared/AutoBeanFactory.java:85: * The
types specified by this annotation will not have be wrapped in by an
Remove "have"

http://gwt-code-reviews.appspot.com/1130801/diff/21001/4013
File
user/src/com/google/gwt/requestfactory/server/RequestFactoryServlet.java
(right):

http://gwt-code-reviews.appspot.com/1130801/diff/21001/4013#newcode98
user/src/com/google/gwt/requestfactory/server/RequestFactoryServlet.java:98:
* @param exceptionHandler an {...@link ExceptionHandler} instance
Missing JavaDoc for serviceDecorators

http://gwt-code-reviews.appspot.com/1130801/diff/21001/4017
File user/src/com/google/gwt/requestfactory/server/ServiceLayer.java
(right):

http://gwt-code-reviews.appspot.com/1130801/diff/21001/4017#newcode69
user/src/com/google/gwt/requestfactory/server/ServiceLayer.java:69: //
Make the ReflectiveServiceLayer point to the cache
ResolverServiceLayer, perhaps?

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

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

Reply via email to