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