Mantis - Squeak Packages
Viewing Issue Advanced Details
7141 Refactoring Browser minor always 08-02-08 14:58 08-02-08 15:07
alexj  
MarcusDenker  
normal  
assigned  
open  
none    
none  
0007141: SLint performs unwanted tests on package extensions.
For instance SLint will produce recommendations pertaining the Stream class if it's used on a package containing some extensions to that class.
 SLint-Extensions-M0007141.cs [^] (232 bytes) 08-02-08 15:05

Notes
(0012431)
alexj   
08-02-08 15:07   
Since SmalllintChecker only analyse those classes that are defined in the environment, the submitted patch overrides '#definesClass:' in PackageEnvironment to fix the problem.

I'm not sure this is the way to go though. In my image '#definesClass:' only has one definition which is to return true. This doesn't make the intended behavior very clear.

The linter should still check extension methods for the selected package but not the classes in which they are defined.


btw I'm running Code Critics from an OB package environment.