TouchInstinct
/
checkstyle
307
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

activation and suppression comments for PMD design rules. Issue #744

This commit is contained in:
Roman Ivanov 2015-04-25 23:46:48 -07:00
parent 55b266cd92
commit cb5f9827e5
1 changed files with 3 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
config/pmd.xml
View File

@ -115,29 +115,20 @@
<exclude name="ImmutableField"/>
<!-- this rule does not have any option, unreasonable to use -->
<exclude name="MissingBreakInSwitch"/>
<exclude name="SimplifyConditional"/>
<!-- we need compare by ref as Tree structure is immutable, we can easily rely on refs -->
<exclude name="CompareObjectsWithEquals"/>
<exclude name="PositionLiteralsFirstInComparisons"/>
<exclude name="PositionLiteralsFirstInCaseInsensitiveComparisons"/>
<exclude name="UnnecessaryLocalBeforeReturn"/>
<exclude name="NonThreadSafeSingleton"/>
<exclude name="UncommentedEmptyMethod"/>
<exclude name="UncommentedEmptyConstructor"/>
<exclude name="AvoidConstantsInterface"/>
<exclude name="UnsynchronizedStaticDateFormatter"/>
<exclude name="PreserveStackTrace"/>
<exclude name="UseCollectionIsEmpty"/>
<exclude name="ClassWithOnlyPrivateConstructorsShouldBeFinal"/>
<exclude name="EmptyMethodInAbstractClassShouldBeAbstract"/>
<exclude name="SingularField"/>
<exclude name="ReturnEmptyArrayRatherThanNull"/>
<exclude name="AbstractClassWithoutAnyMethod"/>
<exclude name="TooFewBranchesForASwitchStatement"/>
<exclude name="LogicInversion"/>
<exclude name="UseVarargs"/>
<!-- we will use our own declaration order logic -->
<exclude name="FieldDeclarationsShouldBeAtStartOfClass"/>
<!-- too much alarms of Checks, we will never move logic out of Check, each Check is independed logic contianer -->
<exclude name="GodClass"/>
<exclude name="AvoidProtectedMethodInFinalClassNotExtending"/>
</rule>
<rule ref="rulesets/java/design.xml/AvoidDeeplyNestedIfStmts">
<properties>