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

Reply via email to