Me too! Both actorState and player in MorphExtension are now unused btw, if you decided to put them there you might as well remove those two at the same time :) Cheers, Henry On Sep 16, 2009, at 4:43 43PM, Marcus Denker wrote:
On 16.09.2009, at 10:29, Gary Chambers wrote:
I get a different "top offenders" list of course. I've managed to refactor the worst offenders.
Benchmarks for the "opening system browser" below show an improvement.
Going from patched to unpatched shows that the unpatched takes between 15% and 20% longer to complete depending on fonts/theme.
(Benchmark code)
| saveMorphs time | [saveMorphs := World submorphs. World removeAllMorphs."heh, heh" time := [1 to: 10 do: [:i | MorphicToolBuilder new open: (Browser new setClass: SystemDictionary selector: nil). World doOneCycleNow]. World submorphs do: [:m | m delete. World doOneCycleNow]. ] timeToRun] ensure:[World addAllMorphs: saveMorphs]. SystemWindow noteTopWindowIn: World. time
(end of benchmark code)
I can make a changeset available for the brave who want to try.
Yes, I would like to try!
Marcus
-- Marcus Denker - http://marcusdenker.de PLEIAD Lab - Computer Science Department (DCC) - University of Chile
_______________________________________________ Pharo-project mailing list Pharo-project@lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project