LGTM
http://gwt-code-reviews.appspot.com/1457801/diff/4001/user/src/com/google/gwt/logging/server/StackTraceDeobfuscator.java File user/src/com/google/gwt/logging/server/StackTraceDeobfuscator.java (right): http://gwt-code-reviews.appspot.com/1457801/diff/4001/user/src/com/google/gwt/logging/server/StackTraceDeobfuscator.java#newcode145 user/src/com/google/gwt/logging/server/StackTraceDeobfuscator.java:145: if (lineNumber == -1) { Sorry, one more thought: I think it would be good to extract -1 into a static final, say LINE_NUMBER_UNKNOWN, similar what's in StackTraceCreator.java http://gwt-code-reviews.appspot.com/1457801/ -- http://groups.google.com/group/Google-Web-Toolkit-Contributors
