LGTM. With nits

http://gwt-code-reviews.appspot.com/1288802/diff/3001/4001
File
user/src/com/google/gwt/validation/rebind/GwtSpecificValidatorCreator.java
(right):

http://gwt-code-reviews.appspot.com/1288802/diff/3001/4001#newcode82
user/src/com/google/gwt/validation/rebind/GwtSpecificValidatorCreator.java:82:
*/
Remove

http://gwt-code-reviews.appspot.com/1288802/diff/3001/4001#newcode309
user/src/com/google/gwt/validation/rebind/GwtSpecificValidatorCreator.java:309:
} catch (NoSuchFieldException ignore) {
Document as
// Expected case

http://gwt-code-reviews.appspot.com/1288802/diff/3001/4001#newcode315
user/src/com/google/gwt/validation/rebind/GwtSpecificValidatorCreator.java:315:
} catch (NoSuchMethodException ignore) {
Same as above

http://gwt-code-reviews.appspot.com/1288802/diff/3001/4001#newcode393
user/src/com/google/gwt/validation/rebind/GwtSpecificValidatorCreator.java:393:
} catch (NoSuchMethodException ignore) {
// Expected case

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

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

Reply via email to