I just went ahead and added test coverage. Meh. No point in caching the field in GenerateJavaAST, it's going away soon. I don't remember why I didn't copy the hack, I think I was trying to be more clever about it to not have to use reflection, but it broke down in this case.
http://gwt-code-reviews.appspot.com/1450814/diff/1/dev/core/src/com/google/gwt/dev/jjs/impl/GwtAstBuilder.java File dev/core/src/com/google/gwt/dev/jjs/impl/GwtAstBuilder.java (right): http://gwt-code-reviews.appspot.com/1450814/diff/1/dev/core/src/com/google/gwt/dev/jjs/impl/GwtAstBuilder.java#newcode2776 dev/core/src/com/google/gwt/dev/jjs/impl/GwtAstBuilder.java:2776: InternalCompilerException.preload(); You mean the InternalCompilerException.preload() call? The target method has spot-on Javadoc already. http://gwt-code-reviews.appspot.com/1450814/ -- http://groups.google.com/group/Google-Web-Toolkit-Contributors
