Hopefully I've not missed anything... I have noticed that a fileout from the change sorter doesn't appear to write out any methods that are in *-override categories... at least that's how it seems (might be another problem!). Regards, Gary ----- Original Message ----- From: "Henrik Johansen" <henrik.s.johansen@veloxit.no> To: <Pharo-project@lists.gforge.inria.fr> Sent: Wednesday, September 16, 2009 3:56 PM Subject: Re: [Pharo-project] Morph extension
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
_______________________________________________ Pharo-project mailing list Pharo-project@lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project