LGTM. With a nit.

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

http://gwt-code-reviews.appspot.com/1369810/diff/1/user/src/com/google/gwt/validation/client/impl/NodeImpl.java#newcode72
user/src/com/google/gwt/validation/client/impl/NodeImpl.java:72: &&
this.iterable == that.iterable;
I find a little worrisome that the equality comparison uses == instead
of equals() for name and index.

http://gwt-code-reviews.appspot.com/1369810/

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

Reply via email to