Fwd: Re: code Critics 'don't check again' optiion? (was [Newbies] isKindOf considered questionable)

squeak414 at free.fr squeak414 at free.fr
Wed Jul 23 08:05:11 UTC 2008

----- Forwarded message from Lukas Renggli <renggli at gmail.com> -----
    Date: Mon, 21 Jul 2008 12:25:47 +0200
    From: Lukas Renggli <renggli at gmail.com>
Reply-To: Lukas Renggli <renggli at gmail.com>
 Subject: Re: code Critics 'don't check again' option? (was [Newbies] isKindOf
considered questionable)
      To: "A friendly place to get answers to even the most basic questions
about Squeak." <beginners at lists.squeakfoundation.org>, squeak414 at free.fr, Marcus
Denker <denker at iam.unibe.ch>

>  If there is interest, I could include the basic functionality into RB
>  itself. I think that would be much cleaner and easier to use.

The latest RB (Refactoring-Core-lr.5.mcz) integrates the functionality
to ignore certain SmallLint rules in particular methods. Contrary to
the original RB implementation it uses pragmas that are persistent
with the method. The original RB used a global dictionary that was not
tracked by versioning systems.

If the rule 'Messages sent but not implemented' should be ignored for
a particular method add the following pragma:

	<lint: 'Messages sent but not implemented'>

Normally you also want to add some more information about your rationale:

	<lint: 'Messages sent but not implemented' rationale: 'We are doing
metaprogramming trickery here' author: 'Lukas Renggli'>

Contrary to my original implementation in the Essential package don't
put a symbol to identify the rule (that doesn't work without changing
a whole lot of RB), but just the title that also appears in the result

Furthermore I added several new rules to Lint: This is to detect not
categorized methods, violations to the Law of Demeter and
unconditional recursion. These rules were also formerly part of the
Essential package and can now be used from the standard RB tools.


Lukas Renggli

----- End forwarded message -----

Hi Damien,

Will this automatically be picked up in your dev images? I for one think it will
be a really useful addition.


More information about the Beginners mailing list