http://gwt-code-reviews.appspot.com/1461803/diff/5004/dev/core/src/com/google/gwt/dev/javac/CompilationUnitBuilder.java
File dev/core/src/com/google/gwt/dev/javac/CompilationUnitBuilder.java
(right):

http://gwt-code-reviews.appspot.com/1461803/diff/5004/dev/core/src/com/google/gwt/dev/javac/CompilationUnitBuilder.java#newcode140
dev/core/src/com/google/gwt/dev/javac/CompilationUnitBuilder.java:140:
Util.copy(in, out);
If you're in to gratuitous renaming for clarity's sake, I think
Util.copyAndClose would be a better name for Util.copy, as it would
signal that you don't need the finally here.

Contrasts nicely with the existing Util.copyNoClose as well.

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

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

Reply via email to