PDM design rules are expanded. Issue #744
This commit is contained in:
parent
4d5d156c07
commit
5a1b933da2
|
|
@ -106,7 +106,62 @@
|
|||
</properties>
|
||||
</rule>
|
||||
|
||||
<!--<rule ref="rulesets/java/design.xml"/>-->
|
||||
<rule ref="rulesets/java/design.xml">
|
||||
<exclude name="UseUtilityClass"/>
|
||||
<exclude name="SimplifyBooleanReturns"/>
|
||||
<exclude name="SimplifyBooleanExpressions"/>
|
||||
<exclude name="SwitchStmtsShouldHaveDefault"/>
|
||||
<exclude name="AvoidDeeplyNestedIfStmts"/>
|
||||
<exclude name="AvoidReassigningParameters"/>
|
||||
<exclude name="SwitchDensity"/>
|
||||
<exclude name="ConstructorCallsOverridableMethod"/>
|
||||
<exclude name="AccessorClassGeneration"/>
|
||||
<exclude name="FinalFieldCouldBeStatic"/>
|
||||
<exclude name="CloseResource"/>
|
||||
<exclude name="NonStaticInitializer"/>
|
||||
<exclude name="DefaultLabelNotLastInSwitchStmt"/>
|
||||
<exclude name="NonCaseLabelInSwitchStatement"/>
|
||||
<exclude name="OptimizableToArrayCall"/>
|
||||
<exclude name="BadComparison"/>
|
||||
<exclude name="EqualsNull"/>
|
||||
<exclude name="ConfusingTernary"/>
|
||||
<exclude name="InstantiationToGetClass"/>
|
||||
<exclude name="IdempotentOperations"/>
|
||||
<exclude name="SimpleDateFormatNeedsLocale"/>
|
||||
<exclude name="ImmutableField"/>
|
||||
<exclude name="UseLocaleWithCaseConversions"/>
|
||||
<exclude name="AvoidProtectedFieldInFinalClass"/>
|
||||
<exclude name="AssignmentToNonFinalStatic"/>
|
||||
<exclude name="MissingStaticMethodInNonInstantiatableClass"/>
|
||||
<exclude name="AvoidSynchronizedAtMethodLevel"/>
|
||||
<exclude name="MissingBreakInSwitch"/>
|
||||
<exclude name="UseNotifyAllInsteadOfNotify"/>
|
||||
<exclude name="AvoidInstanceofChecksInCatchClause"/>
|
||||
<exclude name="AbstractClassWithoutAbstractMethod"/>
|
||||
<exclude name="SimplifyConditional"/>
|
||||
<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"/>
|
||||
<exclude name="FieldDeclarationsShouldBeAtStartOfClass"/>
|
||||
<exclude name="GodClass"/>
|
||||
<exclude name="AvoidProtectedMethodInFinalClassNotExtending"/>
|
||||
</rule>
|
||||
<rule ref="rulesets/java/empty.xml"/>
|
||||
<rule ref="rulesets/java/finalizers.xml"/>
|
||||
<rule ref="rulesets/java/imports.xml"/>
|
||||
|
|
|
|||
Loading…
Reference in New Issue