|Anonymous | Login||07-06-2020 20:58 UTC|
|Main | My View | View Issues | Change Log | Docs|
|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|
|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.|
|Attached Files||SLint-Extensions-M0007141.cs [^] (232 bytes) 08-02-08 15:05|
(0012431 - 542 - 596 - 596 - 596 - 596 - 596)
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.
|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.