LGTM + minor nits

http://gwt-code-reviews.appspot.com/1357801/diff/2001/2004
File dev/core/src/com/google/gwt/dev/javac/Dependencies.java (right):

http://gwt-code-reviews.appspot.com/1357801/diff/2001/2004#newcode41
dev/core/src/com/google/gwt/dev/javac/Dependencies.java:41: class
Dependencies implements Serializable {
80 chars

http://gwt-code-reviews.appspot.com/1357801/diff/2001/2005
File dev/core/src/com/google/gwt/dev/javac/JsniCollector.java (right):

http://gwt-code-reviews.appspot.com/1357801/diff/2001/2005#newcode71
dev/core/src/com/google/gwt/dev/javac/JsniCollector.java:71:
80 chars

http://gwt-code-reviews.appspot.com/1357801/diff/2001/2005#newcode86
dev/core/src/com/google/gwt/dev/javac/JsniCollector.java:86: }
white space

http://gwt-code-reviews.appspot.com/1357801/diff/2001/2009
File dev/core/test/com/google/gwt/dev/javac/CompilationStateTest.java
(right):

http://gwt-code-reviews.appspot.com/1357801/diff/2001/2009#newcode340
dev/core/test/com/google/gwt/dev/javac/CompilationStateTest.java:340:
rebuildCompilationState();
I'm not sure the reformatting on these is better than before?

http://gwt-code-reviews.appspot.com/1357801/show

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

Reply via email to