LGTM. with nits

http://gwt-code-reviews.appspot.com/1282801/diff/1/4
File user/src/com/google/gwt/validation/client/impl/PathImpl.java
(right):

http://gwt-code-reviews.appspot.com/1282801/diff/1/4#newcode44
user/src/com/google/gwt/validation/client/impl/PathImpl.java:44:
nodes.addAll(originalPath.nodes);
Indentation

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

http://gwt-code-reviews.appspot.com/1282801/diff/1/7#newcode279
user/src/com/google/gwt/validation/rebind/GwtSpecificValidatorCreator.java:279:
Class<? extends Annotation> expectedAnnotaionClass) {
Annotaion -> Annotation

http://gwt-code-reviews.appspot.com/1282801/diff/1/7#newcode297
user/src/com/google/gwt/validation/rebind/GwtSpecificValidatorCreator.java:297:
Class<? extends Annotation> expectedAnnotaionClass =
Annotaion -> Annotation

http://gwt-code-reviews.appspot.com/1282801/diff/1/7#newcode307
user/src/com/google/gwt/validation/rebind/GwtSpecificValidatorCreator.java:307:
// TODO(nchalko) check stuff
Will you need some more detail a week from today?

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

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

Reply via email to