Reviewers: scottb, jbrosenberg,

Description:
Patch for interim state of incremental compile.
If a user manually switches GwtAstBuilder on/off, cached units may
not have the AST partial data.


Please review this at http://gwt-code-reviews.appspot.com/1464804/

Affected files:
  M dev/core/src/com/google/gwt/dev/javac/CompilationStateBuilder.java


Index: dev/core/src/com/google/gwt/dev/javac/CompilationStateBuilder.java
===================================================================
--- dev/core/src/com/google/gwt/dev/javac/CompilationStateBuilder.java (revision 10380) +++ dev/core/src/com/google/gwt/dev/javac/CompilationStateBuilder.java (working copy)
@@ -278,6 +278,13 @@
               }
             }
           }
+          if (isValid && GwtAstBuilder.ENABLED) {
+ // TODO(zundel): temporary fix to invalidate a unit when GwtAstBuilder is flipped off
+            // then on again by user.
+ if (unit.getTypesSerialized() == null || unit.getTypesSerialized().length == 0) {
+              isValid = false;
+            }
+          }
           if (!isValid) {
             if (logger.isLoggable(TreeLogger.TRACE)) {
logger.log(TreeLogger.TRACE, "Invalid Unit: " + unit.getTypeName());


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

Reply via email to