Mantis Bugtracker
  

Viewing Issue Simple Details Jump to Notes ] View Advanced ] Issue History ] Print ]
ID Category Severity Reproducibility Date Submitted Last Update
0007141 [Squeak Packages] Refactoring Browser minor always 08-02-08 14:58 08-02-08 15:07
Reporter alexj View Status public  
Assigned To MarcusDenker
Priority normal Resolution open  
Status assigned  
Summary 0007141: SLint performs unwanted tests on package extensions.
Description For instance SLint will produce recommendations pertaining the Stream class if it's used on a package containing some extensions to that class.
Additional Information
Attached Files  SLint-Extensions-M0007141.cs [^] (232 bytes) 08-02-08 15:05

- Relationships

- Notes
(0012431 - 542 - 596 - 596 - 596 - 596 - 596)
alexj
08-02-08 15:07
edited on: 08-02-08 15:21

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.

 

- Issue History
Date Modified Username Field Change
08-02-08 14:58 alexj New Issue
08-02-08 14:58 alexj Status new => assigned
08-02-08 14:58 alexj Assigned To  => MarcusDenker
08-02-08 15:05 alexj File Added: SLint-Extensions-M0007141.cs
08-02-08 15:07 alexj Note Added: 0012431
08-02-08 15:09 alexj Note Edited: 0012431
08-02-08 15:16 alexj Note Edited: 0012431
08-02-08 15:21 alexj Note Edited: 0012431


Mantis 1.0.8[^]
Copyright © 2000 - 2007 Mantis Group
45 total queries executed.
29 unique queries executed.
Powered by Mantis Bugtracker