2015-03-09 14:43 GMT+01:00 Thierry Goubier <[email protected]>:
> > > 2015-03-09 14:39 GMT+01:00 Nicolai Hess <[email protected]>: > >> 2015-03-09 14:18 GMT+01:00 Thierry Goubier <[email protected]>: >> >>> in short, implementorsOf:/referencesTo:/ is broken (does not search over >>> the class side of things). >>> >>> Moreover, depending on the kind of RBEnvironment one is looking at: >>> >>> allClasses >>> returns only the classes >>> or >>> returns classes and metaclasses >>> >>> >> I reported a couple of issues at fogbugz related to this (actually about >> smalllint validations rules, but they >> depend on this change on RBEnvironments) >> And I raised some topics about this on the mailing list. >> > > Ok, then we know what was the reason. > > >> >> I was not involved in the latest changes on RBEnvironment and don't have >> any project that >> depends on RBEnvironment. >> >> So I can *do* the fixes I proposed, but I will not make the decisions. >> Just waiting for feedback. >> > > Which issues are we talking about? I can have a look. > 14230 <https://pharo.fogbugz.com/default.asp?14230> Some rules aren't applied on the classside anymore 14621 <https://pharo.fogbugz.com/default.asp?14621> Some RBBrowserEnvironments don't work well with Metaclasses closed but related: 14228 <https://pharo.fogbugz.com/default.asp?14228> critic rules running on all methods of a class extended by a package (closed) 15077 <https://pharo.fogbugz.com/default.asp?15077> CriticBrowser MNU Trait>>methodClass (closed) closed, related and part of the problem: 14096 <https://pharo.fogbugz.com/default.asp?14096> Rewrite lint rules to check classes and methods 14098 <https://pharo.fogbugz.com/default.asp?14098> allClassesAndTraits significantly different from allClassesAndTraitsDo: And mailing list "discussions" http://forum.world.st/RBEnvironments-allClasses-Traits-Meta-tp4796370.html http://forum.world.st/What-is-a-Behavior-are-Traits-behavior-too-tp4804691.html > > >> >> >>> Of course, implementorsOf: is broken because allClasses does not returns >>> the metaclasses. >>> >>> I like it when people "normalize" things :( >>> >> >> He asked for feedback on this. And as far as I know no one complained. >> > > Fair enough. > > Really need to have more test cases to cover things like that :) > RBEnvironments are so nice, small but really powerfull (refactoring/searching/validation). I would like to get this right. > > Thierry >
