Code Mentor in 6.02

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

Code Mentor in 6.02

Ian Bartholomew-21
The following rules are wrong in the CodeMentor (nothing in explanation
view)

Possible bugs>>Subclass of collection that has instance variable but
doesn't define copyEmpty

lintrules.xml uses "copyEmpty:"

------

Miscellaneous>>Redundant class name in selector

lintrules.xml uses "class summary"

------

Miscellaneous>>Refers to class name instead of "self class"

lintrules.xml uses "class summary"

------

The following rule doesn't display the name of erroneous classes

Unnecessary code>>Unreferenced instance variables


--
Ian

Use the Reply-To address to contact me (limited validity).
Mail sent to the From address is ignored.


Reply | Threaded
Open this post in threaded view
|

Re: Code Mentor in 6.02

Blair McGlashan-3
"Ian Bartholomew" <[hidden email]> wrote in message
news:[hidden email]...
> The following rules are wrong in the CodeMentor (nothing in explanation
> view)
>....

Thanks Ian, you've been busy :-)

These will be sorted in the next patch level.

Regards

Blair


Reply | Threaded
Open this post in threaded view
|

Re: Code Mentor in 6.02

Ian Bartholomew-21
Blair,

> Thanks Ian, you've been busy :-)

Now I've learnt how to drive it properly I find the CodeMentor quite
useful.  It seems quite good at indicating code smells, not necessarily
connected with the flagged error or warning.  I do find that I do have
to restrain myself from diving in with changes to quickly though :-)

Can I add one more that I (re)noticed ...

------

The following rule doesn't display the name of erroneous variables

Unnecessary code>>Instance variables not read AND written

------

--
Ian

Use the Reply-To address to contact me (limited validity).
Mail sent to the From address is ignored.