From 138c2ccc4cc167cdba74961b4d0553f3b728ba4e Mon Sep 17 00:00:00 2001
From: Roman Ivanov <ivanov-jr@mail.ru>
Date: Wed, 22 Apr 2015 16:35:49 -0400
Subject: [PATCH] explnations for exclusions for PMD controversial rules. Issue
 #744

---
 config/pmd.xml | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/config/pmd.xml b/config/pmd.xml
index 0dd0b3c6c..3a42cba15 100644
--- a/config/pmd.xml
+++ b/config/pmd.xml
@@ -60,7 +60,9 @@
   <rule ref="rulesets/java/controversial.xml">
     <!-- calling super() is completely pointless, no matter if class inherits anything or not; it is meaningful only if you do not call implicit constructor of base class -->
     <exclude name="CallSuperInConstructor"/>
+    <!-- We reuse Check instances between java files, we need to clear state of class in beginTree() methods -->
     <exclude name="NullAssignment"/>
+    <!-- it is possible only in functional languages and fanatically-pristine code, withot aditional option that are done at ReturnCountExtendedCheck it is not good rule -->
     <exclude name="OnlyOneReturn"/>
     <exclude name="AssignmentInOperand"/>
     <exclude name="AtLeastOneConstructor"/>