LGTM

Just a couple extra newlines that should be removed before committing.


http://gwt-code-reviews.appspot.com/111805/diff/10/1013
File user/src/com/google/gwt/user/client/ui/HasHorizontalAlignment.java
(right):

http://gwt-code-reviews.appspot.com/111805/diff/10/1013#newcode41
Line 41: *
Extra carriage return not needed.

http://gwt-code-reviews.appspot.com/111805/diff/10/1014
File user/src/com/google/gwt/user/client/ui/HasVerticalAlignment.java
(right):

http://gwt-code-reviews.appspot.com/111805/diff/10/1014#newcode17
Line 17:
Extra carriage return.  This should be a checkstyle warning.

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

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

Reply via email to