Hi marcus do you get an idea why we got this slowdown? Stef Le 30/7/15 14:08, Marcus Denker a écrit :
I added an issue:
https://pharo.fogbugz.com/f/cases/16074/Is-RBClassNotReferencedRule-200x-slo...
i think we need to put in some cashing⦠here if we would could all *used* classes once for an enviromentâ¦.
On 29 Jul 2015, at 17:33, Paul DeBruicker <pdebruic@gmail.com <mailto:pdebruic@gmail.com>> wrote:
Paul DeBruicker wrote
Hi Stef,
The slowdown is dependent upon the # of classes in the package. If you use Fuel then you should see a slowdown of 20x. I just have a lot of classes in this package. The source of the slowdown is in the changes to RBSmallLintChecker>>#checkClass: and RBClassNotReferencedRule>>#checkClass: . The old versions use the linter to check if the literal is used anywhere and the new one uses #allCallsOn.
Hope this helps
Paul
and by "use Fuel" I mean:
rule:=RBClassNotReferencedRule new. environment:=RBBrowserEnvironment new forPackageNames: #('Fuel').
AndreasSystemProfiler spyOn:[RBSmalllintChecker runRule: rule onEnvironment: environment]
-- View this message in context: http://forum.world.st/Is-RBClassNotReferencedRule-200x-slower-in-Pharo-4-vs-... Sent from the Pharo Smalltalk Users mailing list archive at Nabble.com <http://Nabble.com>.