Mostly LG.
http://gwt-code-reviews.appspot.com/915802/diff/1/4 File dev/core/src/com/google/gwt/dev/jjs/JJSOptionsImpl.java (right): http://gwt-code-reviews.appspot.com/915802/diff/1/4#newcode119 dev/core/src/com/google/gwt/dev/jjs/JJSOptionsImpl.java:119: public void setDraftCompile(boolean draft) { Should this be killed (and the getter)? We usually make these options classed simple POJOs with no 'derived' state. Any you already killed OptionDraftCompile. http://gwt-code-reviews.appspot.com/915802/diff/1/5 File dev/core/src/com/google/gwt/dev/jjs/JavaToJavaScriptCompiler.java (right): http://gwt-code-reviews.appspot.com/915802/diff/1/5#newcode686 dev/core/src/com/google/gwt/dev/jjs/JavaToJavaScriptCompiler.java:686: protected static void optimizeJs(JsProgram jsProgram, int optimizationLevel) throws InterruptedException { Might as well spec this as (JJSOptions,JsProgram) to match the Java optimize loop and make way for future options. http://gwt-code-reviews.appspot.com/915802/diff/1/6 File dev/core/src/com/google/gwt/dev/util/arg/ArgHandlerDraftCompile.java (left): http://gwt-code-reviews.appspot.com/915802/diff/1/6#oldcode46 dev/core/src/com/google/gwt/dev/util/arg/ArgHandlerDraftCompile.java:46: optimizeOption.setAggressivelyOptimize(false); This used to explicitly turn OFF aggressive optimizations, but I don't think it does anymore. Is the change in behavior intentional? http://gwt-code-reviews.appspot.com/915802/diff/1/6 File dev/core/src/com/google/gwt/dev/util/arg/ArgHandlerDraftCompile.java (right): http://gwt-code-reviews.appspot.com/915802/diff/1/6#newcode27 dev/core/src/com/google/gwt/dev/util/arg/ArgHandlerDraftCompile.java:27: public <T extends OptionOptimize> ArgHandlerDraftCompile(T option) { No need to templatize anymore, it was just to force two interfaces into one parameter. http://gwt-code-reviews.appspot.com/915802/diff/1/9 File dev/core/src/com/google/gwt/dev/util/arg/OptionOptimize.java (right): http://gwt-code-reviews.appspot.com/915802/diff/1/9#newcode28 dev/core/src/com/google/gwt/dev/util/arg/OptionOptimize.java:28: boolean isDraftCompile(); Any real need for this? http://gwt-code-reviews.appspot.com/915802/show -- http://groups.google.com/group/Google-Web-Toolkit-Contributors
