From 1de66cabc489b18d5aed1a298e3640f5594c57fc Mon Sep 17 00:00:00 2001
From: Richard Cordovano <rcordovano@basistech.com>
Date: Tue, 22 Sep 2020 17:29:51 -0400
Subject: [PATCH] Format Codacy rules file

---
 ruleset.xml | 603 ++++++++++++++++++++++++++--------------------------
 1 file changed, 302 insertions(+), 301 deletions(-)

diff --git a/ruleset.xml b/ruleset.xml
index 0564776c75..b854394891 100644
--- a/ruleset.xml
+++ b/ruleset.xml
@@ -1,307 +1,308 @@
 <?xml version="1.0"?>
 <ruleset name="Autopsy ruleset"
-    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>
-  Ruleset used by Autopsy
-  </description>
-  <rule ref="rulesets/java/basic.xml/SimplifiedTernary"/>
-  <rule ref="rulesets/java/basic.xml/AvoidUsingOctalValues"/>
-  <rule ref="rulesets/java/basic.xml/BigIntegerInstantiation"/>
-  <rule ref="rulesets/java/basic.xml/ClassCastExceptionWithToArray"/>
-  <rule ref="rulesets/java/basic.xml/ForLoopShouldBeWhileLoop"/>
-  <rule ref="rulesets/java/basic.xml/CheckResultSet"/>
-  <rule ref="rulesets/java/basic.xml/UnconditionalIfStatement"/>
-  <rule ref="rulesets/java/basic.xml/CheckSkipResult"/>
-  <rule ref="rulesets/java/basic.xml/DontUseFloatTypeForLoopIndices"/>
-  <rule ref="rulesets/java/basic.xml/MisplacedNullCheck"/>
-  <rule ref="rulesets/java/basic.xml/AvoidDecimalLiteralsInBigDecimalConstructor"/>
-  <rule ref="rulesets/java/basic.xml/BooleanInstantiation"/>
-  <rule ref="rulesets/java/basic.xml/ExtendsObject"/>
-  <rule ref="rulesets/java/basic.xml/AvoidBranchingStatementAsLastInLoop"/>
-  <rule ref="rulesets/java/basic.xml/DoubleCheckedLocking"/>
-  <rule ref="rulesets/java/basic.xml/ReturnFromFinallyBlock"/>
-  <rule ref="rulesets/java/basic.xml/AvoidThreadGroup"/>
-  <rule ref="rulesets/java/basic.xml/CollapsibleIfStatements"/>
-  <rule ref="rulesets/java/basic.xml/AvoidUsingHardCodedIP"/>
-  <rule ref="rulesets/java/basic.xml/CheckResultSet"/>
-  <rule ref="rulesets/java/basic.xml/DontCallThreadRun"/>
-  <rule ref="rulesets/java/basic.xml/BrokenNullCheck"/>
-  <rule ref="rulesets/java/basic.xml/AvoidMultipleUnaryOperators"/>
-  <rule ref="rulesets/java/basic.xml/OverrideBothEqualsAndHashcode"/>
-  <rule ref="rulesets/java/basic.xml/JumbledIncrementer"/>
-  <rule ref="rulesets/java/braces.xml/WhileLoopsMustUseBraces"/>
-  <rule ref="rulesets/java/braces.xml/IfStmtsMustUseBraces"/>
-  <rule ref="rulesets/java/braces.xml/IfElseStmtsMustUseBraces"/>
-  <rule ref="rulesets/java/braces.xml/ForLoopsMustUseBraces"/>
-  <rule ref="rulesets/java/clone.xml/ProperCloneImplementation"/>
-  <rule ref="rulesets/java/clone.xml/CloneThrowsCloneNotSupportedException"/>
-  <rule ref="rulesets/java/clone.xml/CloneMethodMustImplementCloneable"/>
-  <rule ref="rulesets/java/clone.xml/CloneMethodReturnTypeMustMatchClassName"/>
-  <rule ref="rulesets/java/clone.xml/CloneMethodMustBePublic"/>
-  <rule ref="rulesets/java/codesize.xml/NPathComplexity"/>
-  <rule ref="rulesets/java/codesize.xml/ExcessiveMethodLength"/>
-  <rule ref="rulesets/java/codesize.xml/ExcessiveParameterList"/>
-  <rule ref="rulesets/java/codesize.xml/ExcessiveClassLength"/>
-  <rule ref="rulesets/java/codesize.xml/CyclomaticComplexity"/>
-  <rule ref="rulesets/java/codesize.xml/StdCyclomaticComplexity"/>
-  <rule ref="rulesets/java/codesize.xml/ModifiedCyclomaticComplexity"/>
-  <rule ref="rulesets/java/codesize.xml/ExcessivePublicCount"/>
-  <rule ref="rulesets/java/codesize.xml/TooManyFields"/>
-  <rule ref="rulesets/java/codesize.xml/NcssMethodCount"/>
-  <rule ref="rulesets/java/codesize.xml/NcssTypeCount"/>
-  <rule ref="rulesets/java/codesize.xml/NcssConstructorCount"/>
-  <rule ref="rulesets/java/codesize.xml/TooManyMethods"/>
-  <rule ref="rulesets/java/comments.xml/CommentRequired">
-    <properties>            
-      <!-- Disabled because we have lots of undocumented fields -->
-      <property name="fieldCommentRequirement" value="Ignored"/>
-      <!-- Disabled because we don't require comments on overrides of public fields -->
-      <property name="protectedMethodCommentRequirement" value="Ignored"/>
-      <property name="publicMethodCommentRequirement" value="Ignored"/>
-      <!--<property name="methodWithOverrideCommentRequirement" value="Unwanted"/>-->
-    </properties>
-  </rule>
-  <!-- Commented out because it was flagged some of our header / copyright comments
-  <rule ref="rulesets/java/comments.xml/CommentSize"/> -->
-  <rule ref="rulesets/java/comments.xml/CommentContent"/>
-  <!-- Commented out because we use default (package) access often and commenting it does not make sense for us
-  <rule ref="rulesets/java/comments.xml/CommentDefaultAccessModifier"/> -->
-  <!--
-    Commented out because they are controversial and we want basics right now.
-  <rule ref="rulesets/java/controversial.xml/NullAssignment"/>
-  <rule ref="rulesets/java/controversial.xml/OnlyOneReturn"/>
-  <rule ref="rulesets/java/controversial.xml/AssignmentInOperand"/>
-  <rule ref="rulesets/java/controversial.xml/AtLeastOneConstructor"/>
-  <rule ref="rulesets/java/controversial.xml/DontImportSun"/>
-  <rule ref="rulesets/java/controversial.xml/SuspiciousOctalEscape"/>
-  <rule ref="rulesets/java/controversial.xml/CallSuperInConstructor"/>
-  <rule ref="rulesets/java/controversial.xml/UnnecessaryParentheses"/>
-  <rule ref="rulesets/java/controversial.xml/DataflowAnomalyAnalysis"/>
-  <rule ref="rulesets/java/controversial.xml/AvoidFinalLocalVariable"/>
-  <rule ref="rulesets/java/controversial.xml/AvoidUsingShortType"/>
-  <rule ref="rulesets/java/controversial.xml/AvoidUsingVolatile"/>
-  <rule ref="rulesets/java/controversial.xml/AvoidUsingNativeCode"/>
-  <rule ref="rulesets/java/controversial.xml/AvoidAccessibilityAlteration"/>
-  <rule ref="rulesets/java/controversial.xml/DoNotCallGarbageCollectionExplicitly"/>
-  <rule ref="rulesets/java/controversial.xml/AvoidPrefixingMethodParameters"/>
-  <rule ref="rulesets/java/controversial.xml/AvoidLiteralsInIfCondition"/>
-  <rule ref="rulesets/java/controversial.xml/UseObjectForClearerAPI"/>
-  <rule ref="rulesets/java/controversial.xml/UseConcurrentHashMap"/>
-  <rule ref="rulesets/java/controversial.xml/OneDeclarationPerLine"/>
-  <rule ref="rulesets/java/controversial.xml/DefaultPackage"/>
-  <rule ref="rulesets/java/controversial.xml/UnnecessaryConstructor"/>
-  -->
-  <rule ref="rulesets/java/coupling.xml/CouplingBetweenObjects"/>
-  <!--<rule ref="rulesets/java/coupling.xml/ExcessiveImports"/>-->
-  <rule ref="rulesets/java/coupling.xml/LooseCoupling"/>
-  <rule ref="rulesets/java/coupling.xml/LoosePackageCoupling"/>
-  <!-- Commented out because we have not enforced this in the past
-  <rule ref="rulesets/java/coupling.xml/LawOfDemeter"/> -->
-  <rule ref="rulesets/java/design.xml/UseUtilityClass"/>
-  <rule ref="rulesets/java/design.xml/AvoidDeeplyNestedIfStmts"/>
-  <rule ref="rulesets/java/design.xml/SwitchDensity"/>
-  <rule ref="rulesets/java/design.xml/ConstructorCallsOverridableMethod"/>
-  <!--<rule ref="rulesets/java/design.xml/AccessorClassGeneration"/>-->
-  <rule ref="rulesets/java/design.xml/FinalFieldCouldBeStatic"/>
-  <rule ref="rulesets/java/design.xml/CloseResource"/>
-  <rule ref="rulesets/java/design.xml/NonStaticInitializer"/>
-  <rule ref="rulesets/java/design.xml/DefaultLabelNotLastInSwitchStmt"/>
-  <rule ref="rulesets/java/design.xml/OptimizableToArrayCall"/>
-  <rule ref="rulesets/java/design.xml/BadComparison"/>
-  <!-- Disabled.  Could enforce in future, but not top priority right now.
-  <rule ref="rulesets/java/design.xml/ConfusingTernary"/>-->
-  <rule ref="rulesets/java/design.xml/InstantiationToGetClass"/>
-  <rule ref="rulesets/java/design.xml/IdempotentOperations"/>
-  <rule ref="rulesets/java/design.xml/SimpleDateFormatNeedsLocale"/>
-  <rule ref="rulesets/java/design.xml/ImmutableField"/>
-  <!-- Disabled.  Could enforce in future, but not top priority right now.
-  <rule ref="rulesets/java/design.xml/UseLocaleWithCaseConversions"/> -->
-  <rule ref="rulesets/java/design.xml/AvoidProtectedFieldInFinalClass"/>
-  <!-- Disabled.  Could enforce in future, but not top priority right now.
-  <rule ref="rulesets/java/design.xml/AvoidSynchronizedAtMethodLevel"/>-->
-  <rule ref="rulesets/java/design.xml/UseNotifyAllInsteadOfNotify"/>
-  <rule ref="rulesets/java/design.xml/AbstractClassWithoutAbstractMethod"/>
-  <rule ref="rulesets/java/design.xml/SimplifyConditional"/>
-  <!-- Disabled.  Could enforce in future, but not top priority right now.
-  <rule ref="rulesets/java/design.xml/PositionLiteralsFirstInCaseInsensitiveComparisons"/>-->
-  <rule ref="rulesets/java/design.xml/UnnecessaryLocalBeforeReturn"/>
-  <rule ref="rulesets/java/design.xml/NonThreadSafeSingleton"/>
-  <rule ref="rulesets/java/design.xml/SingleMethodSingleton"/>
-  <rule ref="rulesets/java/design.xml/SingletonClassReturningNewInstance"/>
-  <rule ref="rulesets/java/design.xml/UncommentedEmptyConstructor"/>
-  <rule ref="rulesets/java/design.xml/AvoidConstantsInterface"/>
-  <rule ref="rulesets/java/design.xml/UnsynchronizedStaticDateFormatter"/>
-  <rule ref="rulesets/java/design.xml/PreserveStackTrace"/>
-  <rule ref="rulesets/java/design.xml/UseCollectionIsEmpty"/>
-  <rule ref="rulesets/java/design.xml/ClassWithOnlyPrivateConstructorsShouldBeFinal"/>
-  <rule ref="rulesets/java/design.xml/EmptyMethodInAbstractClassShouldBeAbstract"/>
-  <rule ref="rulesets/java/design.xml/ReturnEmptyArrayRatherThanNull"/>
-  <rule ref="rulesets/java/design.xml/AbstractClassWithoutAnyMethod"/>
-  <rule ref="rulesets/java/design.xml/TooFewBranchesForASwitchStatement"/>
-  <!-- Commented out because it flagged many of the NetBeans created UI classes that place
+         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>
+        Ruleset used by Autopsy
+    </description>
+    <rule ref="rulesets/java/basic.xml/SimplifiedTernary"/>
+    <rule ref="rulesets/java/basic.xml/AvoidUsingOctalValues"/>
+    <rule ref="rulesets/java/basic.xml/BigIntegerInstantiation"/>
+    <rule ref="rulesets/java/basic.xml/ClassCastExceptionWithToArray"/>
+    <rule ref="rulesets/java/basic.xml/ForLoopShouldBeWhileLoop"/>
+    <rule ref="rulesets/java/basic.xml/CheckResultSet"/>
+    <rule ref="rulesets/java/basic.xml/UnconditionalIfStatement"/>
+    <rule ref="rulesets/java/basic.xml/CheckSkipResult"/>
+    <rule ref="rulesets/java/basic.xml/DontUseFloatTypeForLoopIndices"/>
+    <rule ref="rulesets/java/basic.xml/MisplacedNullCheck"/>
+    <rule ref="rulesets/java/basic.xml/AvoidDecimalLiteralsInBigDecimalConstructor"/>
+    <rule ref="rulesets/java/basic.xml/BooleanInstantiation"/>
+    <rule ref="rulesets/java/basic.xml/ExtendsObject"/>
+    <rule ref="rulesets/java/basic.xml/AvoidBranchingStatementAsLastInLoop"/>
+    <rule ref="rulesets/java/basic.xml/DoubleCheckedLocking"/>
+    <rule ref="rulesets/java/basic.xml/ReturnFromFinallyBlock"/>
+    <rule ref="rulesets/java/basic.xml/AvoidThreadGroup"/>
+    <rule ref="rulesets/java/basic.xml/CollapsibleIfStatements"/>
+    <rule ref="rulesets/java/basic.xml/AvoidUsingHardCodedIP"/>
+    <rule ref="rulesets/java/basic.xml/CheckResultSet"/>
+    <rule ref="rulesets/java/basic.xml/DontCallThreadRun"/>
+    <rule ref="rulesets/java/basic.xml/BrokenNullCheck"/>
+    <rule ref="rulesets/java/basic.xml/AvoidMultipleUnaryOperators"/>
+    <rule ref="rulesets/java/basic.xml/OverrideBothEqualsAndHashcode"/>
+    <rule ref="rulesets/java/basic.xml/JumbledIncrementer"/>
+    <rule ref="rulesets/java/braces.xml/WhileLoopsMustUseBraces"/>
+    <rule ref="rulesets/java/braces.xml/IfStmtsMustUseBraces"/>
+    <rule ref="rulesets/java/braces.xml/IfElseStmtsMustUseBraces"/>
+    <rule ref="rulesets/java/braces.xml/ForLoopsMustUseBraces"/>
+    <rule ref="rulesets/java/clone.xml/ProperCloneImplementation"/>
+    <rule ref="rulesets/java/clone.xml/CloneThrowsCloneNotSupportedException"/>
+    <rule ref="rulesets/java/clone.xml/CloneMethodMustImplementCloneable"/>
+    <rule ref="rulesets/java/clone.xml/CloneMethodReturnTypeMustMatchClassName"/>
+    <rule ref="rulesets/java/clone.xml/CloneMethodMustBePublic"/>
+    <rule ref="rulesets/java/codesize.xml/NPathComplexity"/>
+    <rule ref="rulesets/java/codesize.xml/ExcessiveMethodLength"/>
+    <rule ref="rulesets/java/codesize.xml/ExcessiveParameterList"/>
+    <rule ref="rulesets/java/codesize.xml/ExcessiveClassLength"/>
+    <rule ref="rulesets/java/codesize.xml/CyclomaticComplexity"/>
+    <rule ref="rulesets/java/codesize.xml/StdCyclomaticComplexity"/>
+    <rule ref="rulesets/java/codesize.xml/ModifiedCyclomaticComplexity"/>
+    <rule ref="rulesets/java/codesize.xml/ExcessivePublicCount"/>
+    <rule ref="rulesets/java/codesize.xml/TooManyFields"/>
+    <rule ref="rulesets/java/codesize.xml/NcssMethodCount"/>
+    <rule ref="rulesets/java/codesize.xml/NcssTypeCount"/>
+    <rule ref="rulesets/java/codesize.xml/NcssConstructorCount"/>
+    <rule ref="rulesets/java/codesize.xml/TooManyMethods"/>
+    <rule ref="rulesets/java/comments.xml/CommentRequired">
+        <properties>            
+            <!-- Disabled because we have lots of undocumented fields -->
+            <property name="fieldCommentRequirement" value="Ignored"/>
+            <!-- Disabled because we don't require comments on overrides of public fields -->
+            <property name="protectedMethodCommentRequirement" value="Ignored"/>
+            <property name="publicMethodCommentRequirement" value="Ignored"/>
+            <!--<property name="methodWithOverrideCommentRequirement" value="Unwanted"/>-->
+        </properties>
+    </rule>
+    <!-- Commented out because it was flagged some of our header / copyright comments
+    <rule ref="rulesets/java/comments.xml/CommentSize"/> -->
+    <rule ref="rulesets/java/comments.xml/CommentContent"/>
+    <!-- Commented out because we use default (package) access often and commenting it does not make sense for us
+    <rule ref="rulesets/java/comments.xml/CommentDefaultAccessModifier"/> -->
+    <!--
+      Commented out because they are controversial and we want basics right now.
+    <rule ref="rulesets/java/controversial.xml/NullAssignment"/>
+    <rule ref="rulesets/java/controversial.xml/OnlyOneReturn"/>
+    <rule ref="rulesets/java/controversial.xml/AssignmentInOperand"/>
+    <rule ref="rulesets/java/controversial.xml/AtLeastOneConstructor"/>
+    <rule ref="rulesets/java/controversial.xml/DontImportSun"/>
+    <rule ref="rulesets/java/controversial.xml/SuspiciousOctalEscape"/>
+    <rule ref="rulesets/java/controversial.xml/CallSuperInConstructor"/>
+    <rule ref="rulesets/java/controversial.xml/UnnecessaryParentheses"/>
+    <rule ref="rulesets/java/controversial.xml/DataflowAnomalyAnalysis"/>
+    <rule ref="rulesets/java/controversial.xml/AvoidFinalLocalVariable"/>
+    <rule ref="rulesets/java/controversial.xml/AvoidUsingShortType"/>
+    <rule ref="rulesets/java/controversial.xml/AvoidUsingVolatile"/>
+    <rule ref="rulesets/java/controversial.xml/AvoidUsingNativeCode"/>
+    <rule ref="rulesets/java/controversial.xml/AvoidAccessibilityAlteration"/>
+    <rule ref="rulesets/java/controversial.xml/DoNotCallGarbageCollectionExplicitly"/>
+    <rule ref="rulesets/java/controversial.xml/AvoidPrefixingMethodParameters"/>
+    <rule ref="rulesets/java/controversial.xml/AvoidLiteralsInIfCondition"/>
+    <rule ref="rulesets/java/controversial.xml/UseObjectForClearerAPI"/>
+    <rule ref="rulesets/java/controversial.xml/UseConcurrentHashMap"/>
+    <rule ref="rulesets/java/controversial.xml/OneDeclarationPerLine"/>
+    <rule ref="rulesets/java/controversial.xml/DefaultPackage"/>
+    <rule ref="rulesets/java/controversial.xml/UnnecessaryConstructor"/>
+    -->
+    <rule ref="rulesets/java/coupling.xml/CouplingBetweenObjects"/>
+    <!--<rule ref="rulesets/java/coupling.xml/ExcessiveImports"/>-->
+    <rule ref="rulesets/java/coupling.xml/LooseCoupling"/>
+    <rule ref="rulesets/java/coupling.xml/LoosePackageCoupling"/>
+    <!-- Commented out because we have not enforced this in the past
+    <rule ref="rulesets/java/coupling.xml/LawOfDemeter"/> -->
+    <rule ref="rulesets/java/design.xml/UseUtilityClass"/>
+    <rule ref="rulesets/java/design.xml/AvoidDeeplyNestedIfStmts"/>
+    <rule ref="rulesets/java/design.xml/SwitchDensity"/>
+    <rule ref="rulesets/java/design.xml/ConstructorCallsOverridableMethod"/>
+    <!--<rule ref="rulesets/java/design.xml/AccessorClassGeneration"/>-->
+    <rule ref="rulesets/java/design.xml/FinalFieldCouldBeStatic"/>
+    <rule ref="rulesets/java/design.xml/CloseResource"/>
+    <rule ref="rulesets/java/design.xml/NonStaticInitializer"/>
+    <rule ref="rulesets/java/design.xml/DefaultLabelNotLastInSwitchStmt"/>
+    <rule ref="rulesets/java/design.xml/OptimizableToArrayCall"/>
+    <rule ref="rulesets/java/design.xml/BadComparison"/>
+    <!-- Disabled.  Could enforce in future, but not top priority right now.
+    <rule ref="rulesets/java/design.xml/ConfusingTernary"/>-->
+    <rule ref="rulesets/java/design.xml/InstantiationToGetClass"/>
+    <rule ref="rulesets/java/design.xml/IdempotentOperations"/>
+    <rule ref="rulesets/java/design.xml/SimpleDateFormatNeedsLocale"/>
+    <rule ref="rulesets/java/design.xml/ImmutableField"/>
+    <!-- Disabled.  Could enforce in future, but not top priority right now.
+    <rule ref="rulesets/java/design.xml/UseLocaleWithCaseConversions"/> -->
+    <rule ref="rulesets/java/design.xml/AvoidProtectedFieldInFinalClass"/>
+    <!-- Disabled.  Could enforce in future, but not top priority right now.
+    <rule ref="rulesets/java/design.xml/AvoidSynchronizedAtMethodLevel"/>-->
+    <rule ref="rulesets/java/design.xml/UseNotifyAllInsteadOfNotify"/>
+    <rule ref="rulesets/java/design.xml/AbstractClassWithoutAbstractMethod"/>
+    <rule ref="rulesets/java/design.xml/SimplifyConditional"/>
+    <!-- Disabled.  Could enforce in future, but not top priority right now.
+    <rule ref="rulesets/java/design.xml/PositionLiteralsFirstInCaseInsensitiveComparisons"/>-->
+    <rule ref="rulesets/java/design.xml/UnnecessaryLocalBeforeReturn"/>
+    <rule ref="rulesets/java/design.xml/NonThreadSafeSingleton"/>
+    <rule ref="rulesets/java/design.xml/SingleMethodSingleton"/>
+    <rule ref="rulesets/java/design.xml/SingletonClassReturningNewInstance"/>
+    <rule ref="rulesets/java/design.xml/UncommentedEmptyConstructor"/>
+    <rule ref="rulesets/java/design.xml/AvoidConstantsInterface"/>
+    <rule ref="rulesets/java/design.xml/UnsynchronizedStaticDateFormatter"/>
+    <rule ref="rulesets/java/design.xml/PreserveStackTrace"/>
+    <rule ref="rulesets/java/design.xml/UseCollectionIsEmpty"/>
+    <rule ref="rulesets/java/design.xml/ClassWithOnlyPrivateConstructorsShouldBeFinal"/>
+    <rule ref="rulesets/java/design.xml/EmptyMethodInAbstractClassShouldBeAbstract"/>
+    <rule ref="rulesets/java/design.xml/ReturnEmptyArrayRatherThanNull"/>
+    <rule ref="rulesets/java/design.xml/AbstractClassWithoutAnyMethod"/>
+    <rule ref="rulesets/java/design.xml/TooFewBranchesForASwitchStatement"/>
+    <!-- Commented out because it flagged many of the NetBeans created UI classes that place
     fields at various places in the class
     <rule ref="rulesets/java/design.xml/FieldDeclarationsShouldBeAtStartOfClass"/> -->
-  <rule ref="rulesets/java/design.xml/GodClass"/>
-  <rule ref="rulesets/java/design.xml/AvoidProtectedMethodInFinalClassNotExtending"/>
-  <rule ref="rulesets/java/design.xml/ConstantsInInterface"/>
-  <rule ref="rulesets/java/design.xml/CompareObjectsWithEquals"/>
-  <rule ref="rulesets/java/design.xml/LogicInversion"/>
-  <rule ref="rulesets/java/design.xml/SingularField"/>
-  <rule ref="rulesets/java/design.xml/SimplifyBooleanReturns"/>
-  <rule ref="rulesets/java/design.xml/AvoidInstanceofChecksInCatchClause"/>
-  <rule ref="rulesets/java/design.xml/NonCaseLabelInSwitchStatement"/>
-  <rule ref="rulesets/java/design.xml/NonStaticInitializer"/>
-  <rule ref="rulesets/java/design.xml/AvoidReassigningParameters"/>
-  <rule ref="rulesets/java/design.xml/SwitchStmtsShouldHaveDefault"/>
-  <rule ref="rulesets/java/design.xml/MissingBreakInSwitch"/>
-  <rule ref="rulesets/java/design.xml/EqualsNull"/>
-  <rule ref="rulesets/java/design.xml/UncommentedEmptyMethodBody"/>
-  <!--<rule ref="rulesets/java/design.xml/SimplifyBooleanExpressions"/>-->
-  <rule ref="rulesets/java/design.xml/AssignmentToNonFinalStatic"/>
-  <rule ref="rulesets/java/design.xml/MissingStaticMethodInNonInstantiatableClass"/>
-  <!-- Disabled.  Could enforce in future, but not top priority right now.
-  <rule ref="rulesets/java/design.xml/PositionLiteralsFirstInComparisons"/>-->
-  <rule ref="rulesets/java/empty.xml/EmptyCatchBlock"/>
-  <rule ref="rulesets/java/empty.xml/EmptyWhileStmt"/>
-  <rule ref="rulesets/java/empty.xml/EmptyFinallyBlock"/>
-  <rule ref="rulesets/java/empty.xml/EmptyIfStmt"/>
-  <rule ref="rulesets/java/empty.xml/EmptyStatementNotInLoop"/>
-  <rule ref="rulesets/java/empty.xml/EmptyTryBlock"/>
-  <rule ref="rulesets/java/empty.xml/EmptyStatementBlock"/>
-  <rule ref="rulesets/java/empty.xml/EmptyStaticInitializer"/>
-  <rule ref="rulesets/java/empty.xml/EmptyInitializer"/>
-  <rule ref="rulesets/java/empty.xml/EmptySwitchStatements"/>
-  <rule ref="rulesets/java/empty.xml/EmptySynchronizedBlock"/>
-  <rule ref="rulesets/java/finalizers.xml/FinalizeOnlyCallsSuperFinalize"/>
-  <rule ref="rulesets/java/finalizers.xml/FinalizeOverloaded"/>
-  <rule ref="rulesets/java/finalizers.xml/FinalizeDoesNotCallSuperFinalize"/>
-  <rule ref="rulesets/java/finalizers.xml/FinalizeShouldBeProtected"/>
-  <rule ref="rulesets/java/finalizers.xml/AvoidCallingFinalize"/>
-  <rule ref="rulesets/java/finalizers.xml/EmptyFinalizer"/>
-  <rule ref="rulesets/java/imports.xml/TooManyStaticImports"/>
-  <rule ref="rulesets/java/imports.xml/UnusedImports"/>
-  <rule ref="rulesets/java/imports.xml/ImportFromSamePackage"/>
-  <rule ref="rulesets/java/imports.xml/DuplicateImports"/>
-  <rule ref="rulesets/java/imports.xml/DontImportJavaLang"/>
-  <!-- Disabled because flagging use of Bundle for bundle messages
-  is not desirable since this qualification helps with code completion
-  while inappropriate use of full qualification is otherwise very rare
-  <rule ref="rulesets/java/imports.xml/UnnecessaryFullyQualifiedName"/>-->
-  <!-- Disabled because it generated a lot of errors for non-Beans
-  <rule ref="rulesets/java/javabeans.xml/BeanMembersShouldSerialize"/>-->
-  <rule ref="rulesets/java/javabeans.xml/MissingSerialVersionUID"/>
-  <rule ref="rulesets/java/logging-java.xml/MoreThanOneLogger"/>
-  <rule ref="rulesets/java/logging-java.xml/LoggerIsNotStaticFinal"/>
-  <rule ref="rulesets/java/logging-java.xml/SystemPrintln"/>
-  <rule ref="rulesets/java/logging-java.xml/AvoidPrintStackTrace"/>
-  <!-- Disabled because we don't want If conditions around all Log statements 
-  <rule ref="rulesets/java/logging-java.xml/GuardLogStatementJavaUtil"/> -->
-  <rule ref="rulesets/java/logging-java.xml/InvalidSlf4jMessageFormat"/>
-  <rule ref="rulesets/java/migrating.xml/ReplaceVectorWithList"/>
-  <rule ref="rulesets/java/migrating.xml/ReplaceHashtableWithMap"/>
-  <rule ref="rulesets/java/migrating.xml/ReplaceEnumerationWithIterator"/>
-  <rule ref="rulesets/java/migrating.xml/AvoidEnumAsIdentifier"/>
-  <rule ref="rulesets/java/migrating.xml/AvoidAssertAsIdentifier"/>
-  <rule ref="rulesets/java/migrating.xml/IntegerInstantiation"/>
-  <rule ref="rulesets/java/migrating.xml/ByteInstantiation"/>
-  <rule ref="rulesets/java/migrating.xml/ShortInstantiation"/>
-  <rule ref="rulesets/java/migrating.xml/LongInstantiation"/>
-  <!-- Disabled because it wanted all private static to be caps 
-  <rule ref="rulesets/java/naming.xml/VariableNamingConventions"/> -->
-  <rule ref="rulesets/java/naming.xml/AvoidDollarSigns"/>
-  <!--<rule ref="rulesets/java/naming.xml/AbstractNaming"/>-->
-  <rule ref="rulesets/java/naming.xml/SuspiciousHashcodeMethodName"/>
-  <rule ref="rulesets/java/naming.xml/SuspiciousConstantFieldName"/>
-  <rule ref="rulesets/java/naming.xml/SuspiciousEqualsMethodName"/>
-  <rule ref="rulesets/java/naming.xml/AvoidFieldNameMatchingTypeName"/>
-  <rule ref="rulesets/java/naming.xml/AvoidFieldNameMatchingMethodName"/>
-  <rule ref="rulesets/java/naming.xml/MisleadingVariableName"/>
-  <rule ref="rulesets/java/naming.xml/ShortClassName"/>
-  <rule ref="category/java/codestyle.xml/ClassNamingConventions">
-    <properties>
-        <property name="classPattern" value="[A-Z][a-zA-Z0-9]*" />
-        <property name="abstractClassPattern" value="[A-Z][a-zA-Z0-9]*" />
-        <property name="interfacePattern" value="[A-Z][a-zA-Z0-9]*" />
-        <property name="enumPattern" value="[A-Z][a-zA-Z0-9]*" />
-        <property name="annotationPattern" value="[A-Z][a-zA-Z0-9]*" />
-        <!-- Disabled because it is too restrictive -->
-        <property name="utilityClassPattern" value="Ignored" />
-    </properties>
-  </rule>  <rule ref="rulesets/java/naming.xml/MethodNamingConventions"/>
-  <!--<rule ref="rulesets/java/naming.xml/GenericsNaming"/>-->
-  <rule ref="rulesets/java/naming.xml/NoPackage"/>
-  <rule ref="rulesets/java/naming.xml/MethodWithSameNameAsEnclosingClass"/>
-  <!--<rule ref="rulesets/java/naming.xml/ShortVariable"/>-->
-  <!-- Commented out because clarity trumps brevity, developers can use their own judgement
-  <rule ref="rulesets/java/naming.xml/LongVariable"/>  -->
-  <rule ref="rulesets/java/naming.xml/ShortMethodName"/>
-  <rule ref="rulesets/java/naming.xml/BooleanGetMethodName"/>
-  <rule ref="rulesets/java/naming.xml/PackageCase"/>
-  <!-- Commented out because this is a nice micro-refinement, but too much of our code does not do this, developers can, but don't have to ignore this practice for now
-  <rule ref="rulesets/java/optimizations.xml/LocalVariableCouldBeFinal"/>
-  <rule ref="rulesets/java/optimizations.xml/MethodArgumentCouldBeFinal"/>  -->
-  <!-- Commented out because this is not typical coding practice and it seems of dubious value, e.g., https://stackoverflow.com/questions/17340421/pmd-avoid-instantiating-new-objects-inside-loops/17458503
-  <rule ref="rulesets/java/optimizations.xml/AvoidInstantiatingObjectsInLoops"/>   -->
-  <rule ref="rulesets/java/optimizations.xml/UseArrayListInsteadOfVector"/>
-  <rule ref="rulesets/java/optimizations.xml/SimplifyStartsWith"/>
-  <!-- Disabled.  Could enforce in future, but not top priority right now.
-  <rule ref="rulesets/java/optimizations.xml/UseStringBufferForStringAppends"/>-->
-  <rule ref="rulesets/java/optimizations.xml/UseArraysAsList"/>
-  <rule ref="rulesets/java/optimizations.xml/AvoidArrayLoops"/>
-  <rule ref="rulesets/java/optimizations.xml/UnnecessaryWrapperObjectCreation"/>
-  <rule ref="rulesets/java/optimizations.xml/AddEmptyString"/>
-  <!--<rule ref="rulesets/java/optimizations.xml/RedundantFieldInitializer"/>-->
-  <rule ref="rulesets/java/optimizations.xml/PrematureDeclaration"/>
-  <rule ref="rulesets/java/strictexception.xml/SignatureDeclareThrowsException"/>
-  <rule ref="rulesets/java/strictexception.xml/AvoidCatchingThrowable"/>
-  <rule ref="rulesets/java/strictexception.xml/ExceptionAsFlowControl"/>
-  <rule ref="rulesets/java/strictexception.xml/AvoidCatchingNPE"/>
-  <rule ref="rulesets/java/strictexception.xml/AvoidRethrowingException"/>
-  <rule ref="rulesets/java/strictexception.xml/DoNotExtendJavaLangError"/>
-  <rule ref="rulesets/java/strictexception.xml/DoNotThrowExceptionInFinally"/>
-  <rule ref="rulesets/java/strictexception.xml/AvoidThrowingNewInstanceOfSameException"/>
-  <rule ref="rulesets/java/strictexception.xml/AvoidLosingExceptionInformation"/>
-  <rule ref="rulesets/java/strictexception.xml/AvoidCatchingGenericException"/>
-  <rule ref="rulesets/java/strictexception.xml/AvoidThrowingRawExceptionTypes"/>
-  <rule ref="rulesets/java/strictexception.xml/AvoidThrowingNullPointerException"/>
-  <!--<rule ref="rulesets/java/strings.xml/AvoidDuplicateLiterals"/>-->
-  <!--<rule ref="rulesets/java/strings.xml/InefficientStringBuffering"/>-->
-  <rule ref="rulesets/java/strings.xml/AppendCharacterWithChar"/>
-  <rule ref="rulesets/java/strings.xml/ConsecutiveAppendsShouldReuse"/>
-  <rule ref="rulesets/java/strings.xml/ConsecutiveLiteralAppends"/>
-  <rule ref="rulesets/java/strings.xml/UseIndexOfChar"/>
-  <rule ref="rulesets/java/strings.xml/InefficientEmptyStringCheck"/>
-  <rule ref="rulesets/java/strings.xml/InsufficientStringBufferDeclaration"/>
-  <rule ref="rulesets/java/strings.xml/UselessStringValueOf"/>
-  <rule ref="rulesets/java/strings.xml/StringBufferInstantiationWithChar"/>
-  <rule ref="rulesets/java/strings.xml/AvoidStringBufferField"/>
-  <rule ref="rulesets/java/strings.xml/StringInstantiation"/>
-  <rule ref="rulesets/java/strings.xml/UseEqualsToCompareStrings"/>
-  <rule ref="rulesets/java/strings.xml/StringToString"/>
-  <rule ref="rulesets/java/strings.xml/UseStringBufferLength"/>
-  <rule ref="rulesets/java/strings.xml/UnnecessaryCaseChange"/>
-  <rule ref="rulesets/java/sunsecure.xml/MethodReturnsInternalArray"/>
-  <rule ref="rulesets/java/sunsecure.xml/ArrayIsStoredDirectly"/>
-  <rule ref="rulesets/java/typeresolution.xml/LooseCoupling"/>
-  <rule ref="rulesets/java/typeresolution.xml/CloneMethodMustImplementCloneable"/>
-  <rule ref="rulesets/java/typeresolution.xml/UnusedImports"/>
-  <rule ref="rulesets/java/typeresolution.xml/SignatureDeclareThrowsException"/>
-  <rule ref="rulesets/java/unnecessary.xml/UnnecessaryConversionTemporary"/>
-  <rule ref="rulesets/java/unnecessary.xml/UnnecessaryFinalModifier"/>
-  <rule ref="rulesets/java/unnecessary.xml/UselessOverridingMethod"/>
-  <rule ref="rulesets/java/unnecessary.xml/UnusedNullCheckInEquals"/>
-  <!-- Disabled.  Could enforce in future, but not top priority right now
-  <rule ref="rulesets/java/unnecessary.xml/UselessParentheses"/>-->
-  <rule ref="rulesets/java/unnecessary.xml/UselessQualifiedThis"/>
-  <rule ref="rulesets/java/unnecessary.xml/UselessOperationOnImmutable"/>
-  <rule ref="rulesets/java/unnecessary.xml/UnnecessaryReturn"/>
-  <!--<rule ref="rulesets/java/unusedcode.xml/UnusedModifier"/>-->
-  <rule ref="rulesets/java/unusedcode.xml/UnusedLocalVariable"/>
-  <!-- Commented out because it flagged many event listeners and other interfaces.
-  <rule ref="rulesets/java/unusedcode.xml/UnusedFormalParameter"/> -->
-  <rule ref="rulesets/java/unusedcode.xml/UnusedPrivateField"/>
-  <rule ref="rulesets/java/unusedcode.xml/UnusedPrivateMethod"/>
+    <rule ref="rulesets/java/design.xml/GodClass"/>
+    <rule ref="rulesets/java/design.xml/AvoidProtectedMethodInFinalClassNotExtending"/>
+    <rule ref="rulesets/java/design.xml/ConstantsInInterface"/>
+    <rule ref="rulesets/java/design.xml/CompareObjectsWithEquals"/>
+    <rule ref="rulesets/java/design.xml/LogicInversion"/>
+    <rule ref="rulesets/java/design.xml/SingularField"/>
+    <rule ref="rulesets/java/design.xml/SimplifyBooleanReturns"/>
+    <rule ref="rulesets/java/design.xml/AvoidInstanceofChecksInCatchClause"/>
+    <rule ref="rulesets/java/design.xml/NonCaseLabelInSwitchStatement"/>
+    <rule ref="rulesets/java/design.xml/NonStaticInitializer"/>
+    <rule ref="rulesets/java/design.xml/AvoidReassigningParameters"/>
+    <rule ref="rulesets/java/design.xml/SwitchStmtsShouldHaveDefault"/>
+    <rule ref="rulesets/java/design.xml/MissingBreakInSwitch"/>
+    <rule ref="rulesets/java/design.xml/EqualsNull"/>
+    <rule ref="rulesets/java/design.xml/UncommentedEmptyMethodBody"/>
+    <!--<rule ref="rulesets/java/design.xml/SimplifyBooleanExpressions"/>-->
+    <rule ref="rulesets/java/design.xml/AssignmentToNonFinalStatic"/>
+    <rule ref="rulesets/java/design.xml/MissingStaticMethodInNonInstantiatableClass"/>
+    <!-- Disabled.  Could enforce in future, but not top priority right now.
+    <rule ref="rulesets/java/design.xml/PositionLiteralsFirstInComparisons"/>-->
+    <rule ref="rulesets/java/empty.xml/EmptyCatchBlock"/>
+    <rule ref="rulesets/java/empty.xml/EmptyWhileStmt"/>
+    <rule ref="rulesets/java/empty.xml/EmptyFinallyBlock"/>
+    <rule ref="rulesets/java/empty.xml/EmptyIfStmt"/>
+    <rule ref="rulesets/java/empty.xml/EmptyStatementNotInLoop"/>
+    <rule ref="rulesets/java/empty.xml/EmptyTryBlock"/>
+    <rule ref="rulesets/java/empty.xml/EmptyStatementBlock"/>
+    <rule ref="rulesets/java/empty.xml/EmptyStaticInitializer"/>
+    <rule ref="rulesets/java/empty.xml/EmptyInitializer"/>
+    <rule ref="rulesets/java/empty.xml/EmptySwitchStatements"/>
+    <rule ref="rulesets/java/empty.xml/EmptySynchronizedBlock"/>
+    <rule ref="rulesets/java/finalizers.xml/FinalizeOnlyCallsSuperFinalize"/>
+    <rule ref="rulesets/java/finalizers.xml/FinalizeOverloaded"/>
+    <rule ref="rulesets/java/finalizers.xml/FinalizeDoesNotCallSuperFinalize"/>
+    <rule ref="rulesets/java/finalizers.xml/FinalizeShouldBeProtected"/>
+    <rule ref="rulesets/java/finalizers.xml/AvoidCallingFinalize"/>
+    <rule ref="rulesets/java/finalizers.xml/EmptyFinalizer"/>
+    <rule ref="rulesets/java/imports.xml/TooManyStaticImports"/>
+    <rule ref="rulesets/java/imports.xml/UnusedImports"/>
+    <rule ref="rulesets/java/imports.xml/ImportFromSamePackage"/>
+    <rule ref="rulesets/java/imports.xml/DuplicateImports"/>
+    <rule ref="rulesets/java/imports.xml/DontImportJavaLang"/>
+    <!-- Disabled because flagging use of Bundle for bundle messages
+    is not desirable since this qualification helps with code completion
+    while inappropriate use of full qualification is otherwise very rare
+    <rule ref="rulesets/java/imports.xml/UnnecessaryFullyQualifiedName"/>-->
+    <!-- Disabled because it generated a lot of errors for non-Beans
+    <rule ref="rulesets/java/javabeans.xml/BeanMembersShouldSerialize"/>-->
+    <rule ref="rulesets/java/javabeans.xml/MissingSerialVersionUID"/>
+    <rule ref="rulesets/java/logging-java.xml/MoreThanOneLogger"/>
+    <rule ref="rulesets/java/logging-java.xml/LoggerIsNotStaticFinal"/>
+    <rule ref="rulesets/java/logging-java.xml/SystemPrintln"/>
+    <rule ref="rulesets/java/logging-java.xml/AvoidPrintStackTrace"/>
+    <!-- Disabled because we don't want If conditions around all Log statements 
+    <rule ref="rulesets/java/logging-java.xml/GuardLogStatementJavaUtil"/> -->
+    <rule ref="rulesets/java/logging-java.xml/InvalidSlf4jMessageFormat"/>
+    <rule ref="rulesets/java/migrating.xml/ReplaceVectorWithList"/>
+    <rule ref="rulesets/java/migrating.xml/ReplaceHashtableWithMap"/>
+    <rule ref="rulesets/java/migrating.xml/ReplaceEnumerationWithIterator"/>
+    <rule ref="rulesets/java/migrating.xml/AvoidEnumAsIdentifier"/>
+    <rule ref="rulesets/java/migrating.xml/AvoidAssertAsIdentifier"/>
+    <rule ref="rulesets/java/migrating.xml/IntegerInstantiation"/>
+    <rule ref="rulesets/java/migrating.xml/ByteInstantiation"/>
+    <rule ref="rulesets/java/migrating.xml/ShortInstantiation"/>
+    <rule ref="rulesets/java/migrating.xml/LongInstantiation"/>
+    <!-- Disabled because it wanted all private static to be caps 
+    <rule ref="rulesets/java/naming.xml/VariableNamingConventions"/> -->
+    <rule ref="rulesets/java/naming.xml/AvoidDollarSigns"/>
+    <!--<rule ref="rulesets/java/naming.xml/AbstractNaming"/>-->
+    <rule ref="rulesets/java/naming.xml/SuspiciousHashcodeMethodName"/>
+    <rule ref="rulesets/java/naming.xml/SuspiciousConstantFieldName"/>
+    <rule ref="rulesets/java/naming.xml/SuspiciousEqualsMethodName"/>
+    <rule ref="rulesets/java/naming.xml/AvoidFieldNameMatchingTypeName"/>
+    <rule ref="rulesets/java/naming.xml/AvoidFieldNameMatchingMethodName"/>
+    <rule ref="rulesets/java/naming.xml/MisleadingVariableName"/>
+    <rule ref="rulesets/java/naming.xml/ShortClassName"/>
+    <rule ref="category/java/codestyle.xml/ClassNamingConventions">
+        <properties>
+            <property name="classPattern" value="[A-Z][a-zA-Z0-9]*" />
+            <property name="abstractClassPattern" value="[A-Z][a-zA-Z0-9]*" />
+            <property name="interfacePattern" value="[A-Z][a-zA-Z0-9]*" />
+            <property name="enumPattern" value="[A-Z][a-zA-Z0-9]*" />
+            <property name="annotationPattern" value="[A-Z][a-zA-Z0-9]*" />
+            <!-- Disabled because it is too restrictive -->
+            <property name="utilityClassPattern" value="Ignored" />
+        </properties>
+    </rule>  
+    <rule ref="rulesets/java/naming.xml/MethodNamingConventions"/>
+    <!--<rule ref="rulesets/java/naming.xml/GenericsNaming"/>-->
+    <rule ref="rulesets/java/naming.xml/NoPackage"/>
+    <rule ref="rulesets/java/naming.xml/MethodWithSameNameAsEnclosingClass"/>
+    <!--<rule ref="rulesets/java/naming.xml/ShortVariable"/>-->
+    <!-- Commented out because clarity trumps brevity, developers can use their own judgement
+    <rule ref="rulesets/java/naming.xml/LongVariable"/>  -->
+    <rule ref="rulesets/java/naming.xml/ShortMethodName"/>
+    <rule ref="rulesets/java/naming.xml/BooleanGetMethodName"/>
+    <rule ref="rulesets/java/naming.xml/PackageCase"/>
+    <!-- Commented out because this is a nice micro-refinement, but too much of our code does not do this, developers can, but don't have to ignore this practice for now
+    <rule ref="rulesets/java/optimizations.xml/LocalVariableCouldBeFinal"/>
+    <rule ref="rulesets/java/optimizations.xml/MethodArgumentCouldBeFinal"/>  -->
+    <!-- Commented out because this is not typical coding practice and it seems of dubious value, e.g., https://stackoverflow.com/questions/17340421/pmd-avoid-instantiating-new-objects-inside-loops/17458503
+    <rule ref="rulesets/java/optimizations.xml/AvoidInstantiatingObjectsInLoops"/>   -->
+    <rule ref="rulesets/java/optimizations.xml/UseArrayListInsteadOfVector"/>
+    <rule ref="rulesets/java/optimizations.xml/SimplifyStartsWith"/>
+    <!-- Disabled.  Could enforce in future, but not top priority right now.
+    <rule ref="rulesets/java/optimizations.xml/UseStringBufferForStringAppends"/>-->
+    <rule ref="rulesets/java/optimizations.xml/UseArraysAsList"/>
+    <rule ref="rulesets/java/optimizations.xml/AvoidArrayLoops"/>
+    <rule ref="rulesets/java/optimizations.xml/UnnecessaryWrapperObjectCreation"/>
+    <rule ref="rulesets/java/optimizations.xml/AddEmptyString"/>
+    <!--<rule ref="rulesets/java/optimizations.xml/RedundantFieldInitializer"/>-->
+    <rule ref="rulesets/java/optimizations.xml/PrematureDeclaration"/>
+    <rule ref="rulesets/java/strictexception.xml/SignatureDeclareThrowsException"/>
+    <rule ref="rulesets/java/strictexception.xml/AvoidCatchingThrowable"/>
+    <rule ref="rulesets/java/strictexception.xml/ExceptionAsFlowControl"/>
+    <rule ref="rulesets/java/strictexception.xml/AvoidCatchingNPE"/>
+    <rule ref="rulesets/java/strictexception.xml/AvoidRethrowingException"/>
+    <rule ref="rulesets/java/strictexception.xml/DoNotExtendJavaLangError"/>
+    <rule ref="rulesets/java/strictexception.xml/DoNotThrowExceptionInFinally"/>
+    <rule ref="rulesets/java/strictexception.xml/AvoidThrowingNewInstanceOfSameException"/>
+    <rule ref="rulesets/java/strictexception.xml/AvoidLosingExceptionInformation"/>
+    <rule ref="rulesets/java/strictexception.xml/AvoidCatchingGenericException"/>
+    <rule ref="rulesets/java/strictexception.xml/AvoidThrowingRawExceptionTypes"/>
+    <rule ref="rulesets/java/strictexception.xml/AvoidThrowingNullPointerException"/>
+    <!--<rule ref="rulesets/java/strings.xml/AvoidDuplicateLiterals"/>-->
+    <!--<rule ref="rulesets/java/strings.xml/InefficientStringBuffering"/>-->
+    <rule ref="rulesets/java/strings.xml/AppendCharacterWithChar"/>
+    <rule ref="rulesets/java/strings.xml/ConsecutiveAppendsShouldReuse"/>
+    <rule ref="rulesets/java/strings.xml/ConsecutiveLiteralAppends"/>
+    <rule ref="rulesets/java/strings.xml/UseIndexOfChar"/>
+    <rule ref="rulesets/java/strings.xml/InefficientEmptyStringCheck"/>
+    <rule ref="rulesets/java/strings.xml/InsufficientStringBufferDeclaration"/>
+    <rule ref="rulesets/java/strings.xml/UselessStringValueOf"/>
+    <rule ref="rulesets/java/strings.xml/StringBufferInstantiationWithChar"/>
+    <rule ref="rulesets/java/strings.xml/AvoidStringBufferField"/>
+    <rule ref="rulesets/java/strings.xml/StringInstantiation"/>
+    <rule ref="rulesets/java/strings.xml/UseEqualsToCompareStrings"/>
+    <rule ref="rulesets/java/strings.xml/StringToString"/>
+    <rule ref="rulesets/java/strings.xml/UseStringBufferLength"/>
+    <rule ref="rulesets/java/strings.xml/UnnecessaryCaseChange"/>
+    <rule ref="rulesets/java/sunsecure.xml/MethodReturnsInternalArray"/>
+    <rule ref="rulesets/java/sunsecure.xml/ArrayIsStoredDirectly"/>
+    <rule ref="rulesets/java/typeresolution.xml/LooseCoupling"/>
+    <rule ref="rulesets/java/typeresolution.xml/CloneMethodMustImplementCloneable"/>
+    <rule ref="rulesets/java/typeresolution.xml/UnusedImports"/>
+    <rule ref="rulesets/java/typeresolution.xml/SignatureDeclareThrowsException"/>
+    <rule ref="rulesets/java/unnecessary.xml/UnnecessaryConversionTemporary"/>
+    <rule ref="rulesets/java/unnecessary.xml/UnnecessaryFinalModifier"/>
+    <rule ref="rulesets/java/unnecessary.xml/UselessOverridingMethod"/>
+    <rule ref="rulesets/java/unnecessary.xml/UnusedNullCheckInEquals"/>
+    <!-- Disabled.  Could enforce in future, but not top priority right now
+    <rule ref="rulesets/java/unnecessary.xml/UselessParentheses"/>-->
+    <rule ref="rulesets/java/unnecessary.xml/UselessQualifiedThis"/>
+    <rule ref="rulesets/java/unnecessary.xml/UselessOperationOnImmutable"/>
+    <rule ref="rulesets/java/unnecessary.xml/UnnecessaryReturn"/>
+    <!--<rule ref="rulesets/java/unusedcode.xml/UnusedModifier"/>-->
+    <rule ref="rulesets/java/unusedcode.xml/UnusedLocalVariable"/>
+    <!-- Commented out because it flagged many event listeners and other interfaces.
+    <rule ref="rulesets/java/unusedcode.xml/UnusedFormalParameter"/> -->
+    <rule ref="rulesets/java/unusedcode.xml/UnusedPrivateField"/>
+    <rule ref="rulesets/java/unusedcode.xml/UnusedPrivateMethod"/>
 
 </ruleset>