LGTM. With nits.

http://gwt-code-reviews.appspot.com/1131801/diff/1/4
File
samples/validationtck/test/com/google/gwt/sample/validationtck/constraints/application/TckTestValidator.java
(right):

http://gwt-code-reviews.appspot.com/1131801/diff/1/4#newcode39
samples/validationtck/test/com/google/gwt/sample/validationtck/constraints/application/TckTestValidator.java:39:
Woman.class})
Try to keep formatting stable.
Remove noise (// Visibility.class)

http://gwt-code-reviews.appspot.com/1131801/diff/1/11
File
user/src/com/google/gwt/validation/client/impl/AbstractGwtSpecificValidator.java
(right):

http://gwt-code-reviews.appspot.com/1131801/diff/1/11#newcode76
user/src/com/google/gwt/validation/client/impl/AbstractGwtSpecificValidator.java:76:
*/
fill-in or remove java-doc

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

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

Reply via email to