PMD: Remove rule as it yields too many FP

Author: AB-xdev

.config/pmd/java/ruleset.xml

@@ -766,69 +766,6 @@ public class Foo {
 		</example>
 	</rule>
 
-	<rule name="InitializeComparatorOnlyOnce"
-		language="java"
-		message="Avoid creating Comparator instances repeatedly. Initialize them as static final fields instead."
-		class="net.sourceforge.pmd.lang.rule.xpath.XPathRule">
-		<description>
-			Creating Comparator instances repeatedly in methods like compareTo or sort calls is inefficient.
-
-			Solution: Initialize the Comparator once as a static final field and reuse.
-		</description>
-		<priority>2</priority>
-		<properties>
-			<property name="xpath">
-				<value>
-<![CDATA[
-//MethodCall[
-	(pmd-java:matchesSig('java.util.Comparator#comparing(_*)')
-	or pmd-java:matchesSig('java.util.Comparator#comparingInt(_)')
-	or pmd-java:matchesSig('java.util.Comparator#comparingDouble(_)')
-	or pmd-java:matchesSig('java.util.Comparator#comparingLong(_)')
-	or pmd-java:matchesSig('java.util.Comparator#nullsFirst(_)')
-	or pmd-java:matchesSig('java.util.Comparator#nullsLast(_)')
-	or pmd-java:matchesSig('java.util.Comparator#reversed()')
-	)
-	(: not chained: AST is backwards with first call last element :)
-	and not(MethodCall[pmd-java:typeIs('java.util.Comparator')])
-	(: not used as argument to another Comparator method :)
-	and not(parent::ArgumentList/parent::MethodCall[
-		pmd-java:typeIs('java.util.Comparator')
-	])
-	(: used in non-static-final way :)
-	and not(ancestor::FieldDeclaration[pmd-java:modifiers() = ('final') and pmd-java:modifiers() = ('static')])
-]
-]]>
-				</value>
-			</property>
-		</properties>
-		<example>
-<![CDATA[
-public class BadPerson implements Comparable<Person> {
-	@Override
-	public int compareTo(@NotNull Person o) {
-		return Comparator.comparing(Person::getFirstName)  // Bad: Creates new Comparator instance on each invocation
-				.thenComparing(Person::getLastName)
-				.thenComparingInt(Person::getAge)
-				.compare(this, o);
-	}
-}
-
-public class GoodPerson implements Comparable<Person> {
-	private static final Comparator<Person> COMPARE_FIRST_LAST_NAME_AGE =
-		Comparator.comparing(Person::getFirstName)
-			.thenComparing(Person::getLastName)
-			.thenComparingInt(Person::getAge);
-
-	@Override
-	public int compareTo(@NotNull Person o) {
-		return COMPARE_FIRST_LAST_NAME_AGE.compare(this, o);     // Good: Comparator initialized once as static final field
-	}
-}
-]]>
-		</example>
-	</rule>
-
 	<rule name="AvoidCommonPoolForBlockingCalls" 
 		language="java"
 		message="Avoid the ForkJoinPool::commonPool used in parallelStream for blocking calls"