[ http://jira.codehaus.org/browse/MPPMD-20?page=comments#action_57904 ] 

Sivanandan Srilakshmanan commented on MPPMD-20:
-----------------------------------------------

Please allow me to explain the difficulty I face when using PMD Report.

Firstly
-------
If I need to suppress "OnlyOneReturn" under the Controversial Rules I need to 
specify it as <exclude name="OnlyOneReturnRule"/>

Where as when I need to suppress "CallSuperInConstructor" under the 
Controversial Rules I need to specify it as <exclude 
name="CallSuperInConstructor"/>

As you can see there is no document/consistency on the naming of the actual 
rules.
In "OnlyOneReturn" I need to actually exclude "OnlyOneReturnRule".
On the other hand for CallSuperInConstructor it is not a problem as the rule 
itself is called "CallSuperInConstructor".

Secondly
--------
Controversial Rules is contained in rulesets/controversial.xml Would Security 
Code Guidelines be in rulesets/SecurityCodeGuidelines.xml


If I am incorrect then I am sorry for creating the confusion and would much 
appreciate clarification on the correct usage or a methodical approach to 
finding the rules and rule set.

If I am correct, then I feel that the "rulesets/SecurityCodeGuidelines.xml" 
should be included as part of the heading. For example "Security Code 
Guidelines : rulesets/SecurityCodeGuidelines.xml"
and the rule to be the name itself. ie instead of calling OnlyOneReturn" under 
the Controversial Rules, call it OnlyOneReturnRule.

Thanks
Lakshman


> ruleset names
> -------------
>
>          Key: MPPMD-20
>          URL: http://jira.codehaus.org/browse/MPPMD-20
>      Project: maven-pmd-plugin
>         Type: Improvement

>  Environment: Web browser
>     Reporter: Sivanandan Srilakshmanan
>     Priority: Minor
>      Fix For: 1.8

>
>
> Web page http://pmd.sourceforge.net/rules/index.html
> The PMD index should include the name of the ruleset for example 
> Braces Rules is called rulesets/braces.xml
> Jakarta Commons Logging Rules is called ??

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to