Author: [EMAIL PROTECTED]
Date: Mon Oct  6 11:48:14 2008
New Revision: 3716

Modified:
     
changes/spoon/runAsync/dev/core/src/com/google/gwt/dev/jjs/impl/Pruner.java

Log:
Run visitors over the set of all declared types, not over the whole  
JProgram.
Visiting a JProgram ends up visiting entry methods twice, which breaks
an assumption of CleanUpRefsVisitor.


Modified:  
changes/spoon/runAsync/dev/core/src/com/google/gwt/dev/jjs/impl/Pruner.java
==============================================================================
---  
changes/spoon/runAsync/dev/core/src/com/google/gwt/dev/jjs/impl/Pruner.java     
 
(original)
+++  
changes/spoon/runAsync/dev/core/src/com/google/gwt/dev/jjs/impl/Pruner.java     
 
Mon Oct  6 11:48:14 2008
@@ -468,7 +468,7 @@
        PruneVisitor pruner = new PruneVisitor(
            livenessAnalyzer.getReferencedTypes(),
            livenessAnalyzer.getLiveFieldsAndMethods());
-      pruner.accept(program);
+      pruner.accept(program.getDeclaredTypes());
        if (!pruner.didChange()) {
          break;
        }
@@ -476,7 +476,7 @@
        CleanupRefsVisitor cleaner = new CleanupRefsVisitor(
            livenessAnalyzer.getLiveFieldsAndMethods(),
            pruner.getMethodToOriginalParamsMap());
-      cleaner.accept(program);
+      cleaner.accept(program.getDeclaredTypes());

        madeChanges = true;
      }

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

Reply via email to