Reviewers: spoon, kprobst, scottb,
Description:
This patch incorporates Kathrin's StringBuilder work along with other
miscellaneous speedups found by using JProfiler.
Please review this at http://gwt-code-reviews.appspot.com/34818
Affected files:
Any numbers on the amount of improvement?
On Tue, May 26, 2009 at 11:04 AM, r...@google.com wrote:
Reviewers: spoon, kprobst, scottb,
Description:
This patch incorporates Kathrin's StringBuilder work along with other
miscellaneous speedups found by using JProfiler.
Please review this at
I see around 2x for the story generation phase. It's hard to measure at
the moment because I haven't been able to get a working before build so I
am estimating based on some numbers jotted down earlier.
Dan
On Tue, May 26, 2009 at 2:27 PM, Bruce Johnson br...@google.com wrote:
Any numbers
Author: sco...@google.com
Date: Tue May 26 11:40:15 2009
New Revision: 5471
Modified:
branches/snapshot-2009.05.12-r5406/branch-info.txt
branches/snapshot-2009.05.12-r5406/dev/core/src/com/google/gwt/core/ext/linker/impl/hosted.html
Author: sco...@google.com
Date: Tue May 26 11:56:11 2009
New Revision: 5472
Modified:
trunk/dev/core/src/com/google/gwt/dev/shell/CompilingClassLoader.java
trunk/dev/core/src/com/google/gwt/dev/shell/rewrite/HostedModeClassRewriter.java
Author: sco...@google.com
Date: Tue May 26 12:01:52 2009
New Revision: 5473
Modified:
branches/snapshot-2009.05.12-r5406/branch-info.txt
branches/snapshot-2009.05.12-r5406/dev/core/src/com/google/gwt/dev/shell/CompilingClassLoader.java
Reviewers: bobv,
Please review this at http://gwt-code-reviews.appspot.com/34819
Affected files:
distro-source/core/src/gwt-module.dtd
Index: distro-source/core/src/gwt-module.dtd
===
---
Reviewers: bobv,
Description:
Currently JsInliner.InliningVisitor.endVisit() tries a sequence of
identifiers of the form A_B_number until it finds one that has not
been used in the given scope. Profiling indicates that this loop may
execute hundreds of times for a given name, accounting for a
http://gwt-code-reviews.appspot.com/34820/diff/1/2
File dev/core/src/com/google/gwt/dev/js/JsInliner.java (right):
http://gwt-code-reviews.appspot.com/34820/diff/1/2#newcode262
Line 262: public ComplexityEstimator() { }
Unnecessary constructor.
