LGTM
http://gwt-code-reviews.appspot.com/1519803/diff/1/user/src/com/google/gwt/user/cellview/client/ColumnSortEvent.java File user/src/com/google/gwt/user/cellview/client/ColumnSortEvent.java (right): http://gwt-code-reviews.appspot.com/1519803/diff/1/user/src/com/google/gwt/user/cellview/client/ColumnSortEvent.java#newcode88 user/src/com/google/gwt/user/cellview/client/ColumnSortEvent.java:88: extra spaces http://gwt-code-reviews.appspot.com/1519803/diff/1/user/src/com/google/gwt/user/cellview/client/ColumnSortEvent.java#newcode137 user/src/com/google/gwt/user/cellview/client/ColumnSortEvent.java:137: extra spaces http://gwt-code-reviews.appspot.com/1519803/diff/1/user/src/com/google/gwt/user/cellview/client/ColumnSortEvent.java#newcode138 user/src/com/google/gwt/user/cellview/client/ColumnSortEvent.java:138: public void setList(List<T> list) { You should assert that list is not null. http://gwt-code-reviews.appspot.com/1519803/ -- http://groups.google.com/group/Google-Web-Toolkit-Contributors
