0007141: SLint performs unwanted tests on package extensions.

Viewing Issue Simple Details [ Jump to Notes ]

[ View Advanced ] [ Issue History ] [ Print ]

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

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