blob: 2fa5c3f0e709bfd223e3926228bcc561afb13c06 [file] [log] [blame]
<?xml version="1.0"?>
<!--
Licensed to the Apache Software Foundation (ASF) under one or more
contributor license agreements. See the NOTICE file distributed with
this work for additional information regarding copyright ownership.
The ASF licenses this file to You under the Apache License, Version 2.0
(the "License"); you may not use this file except in compliance with
the License. You may obtain a copy of the License at
http://www.apache.org/licenses/LICENSE-2.0
Unless required by applicable law or agreed to in writing, software
distributed under the License is distributed on an "AS IS" BASIS,
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
See the License for the specific language governing permissions and
limitations under the License.
-->
<ruleset name="commons-statistics-customized"
xmlns="http://pmd.sourceforge.net/ruleset/2.0.0"
xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
xsi:schemaLocation="http://pmd.sourceforge.net/ruleset/2.0.0 http://pmd.sourceforge.net/ruleset_2_0_0.xsd">
<description>
This ruleset checks the code for discouraged programming constructs.
</description>
<rule ref="category/java/bestpractices.xml">
<exclude name="UseVarargs" />
</rule>
<rule ref="category/java/codestyle.xml">
<exclude name="MethodArgumentCouldBeFinal" />
<exclude name="ShortVariable" />
<exclude name="LongVariable" />
<exclude name="CommentDefaultAccessModifier" />
<exclude name="DefaultPackage" />
<exclude name="CallSuperInConstructor" />
<exclude name="AbstractNaming" />
<!-- We do use extra parentheses there as most people do not recall operator precedence,
this means even if the parentheses are useless for the compiler, we don't consider
them useless for the developer. This is the reason why we disable this rule. -->
<exclude name="UselessParentheses" />
<exclude name="AtLeastOneConstructor" />
<exclude name="GenericsNaming" />
<exclude name="OnlyOneReturn" />
<exclude name="UseUnderscoresInNumericLiterals" />
<!-- Disable this rule in favour of the more configurable rule in checkstyle. -->
<exclude name="LocalVariableNamingConventions" />
</rule>
<rule ref="category/java/design.xml">
<exclude name="TooManyMethods" />
<exclude name="LawOfDemeter" />
<exclude name="NcssCount" />
<exclude name="LoosePackageCoupling" />
</rule>
<rule ref="category/java/documentation.xml">
<exclude name="CommentSize" />
</rule>
<rule ref="category/java/errorprone.xml">
<exclude name="BeanMembersShouldSerialize" />
<!-- This rule is known to be poor with Java 5 and later:
https://github.com/pmd/pmd/issues/873 -->
<exclude name="DataflowAnomalyAnalysis" />
</rule>
<rule ref="category/java/multithreading.xml">
<!-- <exclude name="..." /> -->
</rule>
<rule ref="category/java/performance.xml">
<!-- <exclude name="..." /> -->
</rule>
<!-- Rule customisations. -->
<rule ref="category/java/bestpractices.xml/ArrayIsStoredDirectly">
<properties>
<!-- Array is generated internally in this case. -->
<property name="violationSuppressXPath" value="//ClassOrInterfaceDeclaration[@Image='DistributionException']"/>
</properties>
</rule>
<rule ref="category/java/documentation.xml/CommentSize">
<properties>
<property name="maxLines" value="200"/>
<property name="maxLineLength" value="256"/>
</properties>
</rule>
<rule ref="category/java/design.xml/NPathComplexity">
<properties>
<property name="violationSuppressXPath"
value="//ClassOrInterfaceDeclaration[@Image='AbstractContinuousDistribution']"/>
</properties>
</rule>
<rule ref="category/java/design.xml/CyclomaticComplexity">
<properties>
<!-- Increase from default of 10. -->
<property name="methodReportLevel" value="20"/>
</properties>
</rule>
<rule ref="category/java/errorprone.xml/AvoidLiteralsInIfCondition">
<properties>
<!-- Add 1 as a magic number. -->
<property name="ignoreMagicNumbers" value="-1,0,1" />
</properties>
</rule>
</ruleset>