LGTM with nits.

http://gwt-code-reviews.appspot.com/1354807/diff/10/5001
File user/src/com/google/gwt/validation/rebind/BeanHelper.java (right):

http://gwt-code-reviews.appspot.com/1354807/diff/10/5001#newcode52
user/src/com/google/gwt/validation/rebind/BeanHelper.java:52: public
static final Function<BeanHelper, Class<?>> TO_CLAZZ = new
Function<BeanHelper, Class<?>>() {
Line too long

http://gwt-code-reviews.appspot.com/1354807/diff/10/5001#newcode58
user/src/com/google/gwt/validation/rebind/BeanHelper.java:58: private
static final Validator serverSideValidor =
Validation.buildDefaultValidatorFactory().getValidator();
serverSideValidator

http://gwt-code-reviews.appspot.com/1354807/diff/10/5006
File
user/test/org/hibernate/jsr303/tck/tests/validation/ValidateCompileTest.java
(right):

http://gwt-code-reviews.appspot.com/1354807/diff/10/5006#newcode46
user/test/org/hibernate/jsr303/tck/tests/validation/ValidateCompileTest.java:46:
"Unable to create a validator for
org.hibernate.jsr303.tck.tests.validation.Boy becuase "
becuase -> because

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

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

Reply via email to