Re: [Webmacro-devel] Usage of CPD/PMD/Checkstyle

[Alex F. <al...@fi...>]

On Wednesday 23 August 2006 14:42, Marcello H wrote:
> Hello everybody,
>
> I don't know if there are people familiar with CPD?
> (It's a subpackage of PMD.)
> http://pmd.sourceforge.net/cpd.html
>
> CPD is a Copy/Paste Detector.
>
> I started the cpdgui.bat and let it analyze our beloved webmacro-sources.
> It detected 25 pieces of duplicated code.
>
> I also did some work with PMD and Checkstyle.
> Both packages mentioned a lot of hints about our software.
>
> Perhaps something useable for someone to refactor?

I've also been playing around with eclipse and this has been giving me lots of 
warnings about unused variables and the like which are probably worth looking 
at.

However, the majority of the warnings were either in the unit testing trees 
(not so important), or the org.webmacro.parser (WMParser & 
WMParser_implTokenManager) which is just a bit too scary to go poking around 
in.

It would be nice though to have a warning free build from CVS as otherwise 
genuine problems tend to get obscured by legacy issues...

Marcel: can you stick some kind of report about the issues online somewhere 
and I'll see whether eclipse is complaining about similar issues?  I'll see 
if I can find a way of exporting a problems report from eclipse and do the 
same.

Alex