http://gwt-code-reviews.appspot.com/1727807/diff/7/user/src/com/google/gwt/editor/client/impl/SimpleViolation.java File user/src/com/google/gwt/editor/client/impl/SimpleViolation.java (right):
http://gwt-code-reviews.appspot.com/1727807/diff/7/user/src/com/google/gwt/editor/client/impl/SimpleViolation.java#newcode121 user/src/com/google/gwt/editor/client/impl/SimpleViolation.java:121: for (AbstractEditorDelegate<?, ?> baseDelegate : delegateList) { This function is getting kind of long, so you might want to try taking the contents of this for loop and making it a separate function. (But only if we don't end up passing lots of parameters.) http://gwt-code-reviews.appspot.com/1727807/ -- http://groups.google.com/group/Google-Web-Toolkit-Contributors
