LGTM with nits.

http://gwt-code-reviews.appspot.com/1142801/diff/1/8
File user/src/com/google/gwt/validation/rebind/BeanHelper.java (right):

http://gwt-code-reviews.appspot.com/1142801/diff/1/8#newcode78
user/src/com/google/gwt/validation/rebind/BeanHelper.java:78: * @param
bean
Remove @param of fill-in

http://gwt-code-reviews.appspot.com/1142801/diff/1/9
File
user/src/com/google/gwt/validation/rebind/GwtSpecificValidatorCreator.java
(right):

http://gwt-code-reviews.appspot.com/1142801/diff/1/9#newcode57
user/src/com/google/gwt/validation/rebind/GwtSpecificValidatorCreator.java:57:
/**
Fix indentation

http://gwt-code-reviews.appspot.com/1142801/diff/1/9#newcode416
user/src/com/google/gwt/validation/rebind/GwtSpecificValidatorCreator.java:416:
* @param sw
Fill-in or remove

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

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

Reply via email to