http://gwt-code-reviews.appspot.com/1464803/diff/1/dev/core/src/com/google/gwt/dev/CompileModule.java File dev/core/src/com/google/gwt/dev/CompileModule.java (right):
http://gwt-code-reviews.appspot.com/1464803/diff/1/dev/core/src/com/google/gwt/dev/CompileModule.java#newcode189 dev/core/src/com/google/gwt/dev/CompileModule.java:189: Set<String> alreadyLoadedArchives = new HashSet<String>(); On 2011/06/23 14:56:29, jbrosenberg wrote:
So, this change is essentially moving the set of archived units to be
global for
a specific set of modules passed in. I see that this will reduce the
amount of
work needed for a specific set of modules. However, what about the
case where
you then have a different overlapping set of modules. This could then
cause
more work there, since the second set may see that it needs to compile
units
that previously would have been included in the subset of modules that
are
shared between the 2 sets, etc....
This program already has enough pitfalls, so I decided to address this issue. It took a lot more code, but I think it makes it more general. http://gwt-code-reviews.appspot.com/1464803/ -- http://groups.google.com/group/Google-Web-Toolkit-Contributors
