diff --git a/pmd/rulesets/cougaar.xml b/pmd/rulesets/cougaar.xml index 5067853dce..d334a15db5 100644 --- a/pmd/rulesets/cougaar.xml +++ b/pmd/rulesets/cougaar.xml @@ -1,6 +1,6 @@ - + These rules are specific to Cougaar diff --git a/pmd/rulesets/ticbuild.xml b/pmd/rulesets/ticbuild.xml new file mode 100644 index 0000000000..6dd96035f4 --- /dev/null +++ b/pmd/rulesets/ticbuild.xml @@ -0,0 +1,121 @@ + + + + + These are rules which apply to the build at https://cvs.ultralog.net + + + + +Empty Catch Block, will find instances where an exception is caught, +but nothing is done. In most circumstances, this swallows an exception +which should either be acted on or reported. + + + + + + + + + +Empty If Statement, will find instances where a condition is checked, +but nothing is done about it. + + + + + + + + +Empty While Statement, will find all instances where a while statement +does nothing. If it is a timing loop, then you should use Thread.sleep() for it; if +it's a while loop that does a lot in the exit expression, rewrite it to make it clearer. + + + + + + + + + + Avoid using if..else statements without using curly braces + + + + + + + + + + Avoid unnecessary temporaries when converting primitives to Strings + + + + + + + + + + + If you have a class that has nothing but static methods, consider making it a Singleton + + + + + + + + + + diff --git a/pmd/rulesets/unusedcode.xml b/pmd/rulesets/unusedcode.xml index 691bdf800e..74e5cfd9e9 100644 --- a/pmd/rulesets/unusedcode.xml +++ b/pmd/rulesets/unusedcode.xml @@ -1,6 +1,6 @@ - + The Unused Code Ruleset contains a collection of rules that find unused code.