http://gwt-code-reviews.appspot.com/1467803/diff/1/user/test/com/google/gwt/validation/tck/ConstraintsGroupsGwtSuite.java File user/test/com/google/gwt/validation/tck/ConstraintsGroupsGwtSuite.java (right):
http://gwt-code-reviews.appspot.com/1467803/diff/1/user/test/com/google/gwt/validation/tck/ConstraintsGroupsGwtSuite.java#newcode26 user/test/com/google/gwt/validation/tck/ConstraintsGroupsGwtSuite.java:26: * Tck Tests for the {@code constraints groups } package. On 2011/06/22 13:40:52, rchandia wrote:
Extra space
Done. http://gwt-code-reviews.appspot.com/1467803/diff/1/user/test/org/hibernate/jsr303/tck/tests/constraints/groups/DefaultGroupRedefinitionCompileTest.java File user/test/org/hibernate/jsr303/tck/tests/constraints/groups/DefaultGroupRedefinitionCompileTest.java (right): http://gwt-code-reviews.appspot.com/1467803/diff/1/user/test/org/hibernate/jsr303/tck/tests/constraints/groups/DefaultGroupRedefinitionCompileTest.java#newcode39 user/test/org/hibernate/jsr303/tck/tests/constraints/groups/DefaultGroupRedefinitionCompileTest.java:39: builder.expect(TreeLogger.ERROR, // On 2011/06/22 13:40:52, rchandia wrote:
Why the // at EOL?
forces the formatter to do the right thing. http://gwt-code-reviews.appspot.com/1467803/diff/1/user/test/org/hibernate/jsr303/tck/tests/constraints/groups/DefaultGroupRedefinitionCompileTest.java#newcode72 user/test/org/hibernate/jsr303/tck/tests/constraints/groups/DefaultGroupRedefinitionCompileTest.java:72: GroupSequenceWithNoImplicitDefaultGroupValidatorFactory.GroupSequenceWithNoImplicitDefaultGroupValidator.class); On 2011/06/22 13:40:52, rchandia wrote:
Line too long
Renamed, now it fits. http://gwt-code-reviews.appspot.com/1467803/ -- http://groups.google.com/group/Google-Web-Toolkit-Contributors
