[java] TooFewBranchesForSwitch: don't report empty switch

Empty switch statements are already reported
by EmptyControlStatement

docs/pages/release_notes.md

@@ -24,6 +24,8 @@ See [PR #5040](https://github.com/pmd/pmd/pull/5040) for details.
 ### 🌟 Rule Changes
 
 #### Changed Rules
+* {% rule java/performance/TooFewBranchesForSwitch %} (Java Performance) doesn't report empty switches anymore.
+  To detect these, use {% rule java/codestyle/EmptyControlStatement %}.
 * {% rule java/bestpractices/UnitTestShouldUseAfterAnnotation %} (Java Best Practices) now also considers JUnit 5 and TestNG tests.
 * {% rule java/bestpractices/UnitTestShouldUseBeforeAnnotation %} (Java Best Practices) now also considers JUnit 5 and TestNG tests.
 

pmd-java/src/main/resources/category/java/performance.xml

@@ -631,7 +631,7 @@ Note: This rule was named TooFewBranchesForASwitchStatement before PMD 7.7.0.
 //(SwitchStatement | SwitchExpression)
   [ (count(*) - 1) < $minimumNumberCaseForASwitch ]
   (: only consider if no pattern matching is used :)
-  [not(*/SwitchLabel[@PatternLabel = true()])]
+  [*/SwitchLabel[@PatternLabel = false()]]
 ]]>
                 </value>
             </property>

pmd-java/src/test/resources/net/sourceforge/pmd/lang/java/rule/performance/xml/TooFewBranchesForSwitch.xml

@@ -5,13 +5,13 @@
     xsi:schemaLocation="http://pmd.sourceforge.net/rule-tests http://pmd.sourceforge.net/rule-tests_1_0_0.xsd">
 
     <test-code>
-        <description>Switch Statement with no case, not ok</description>
+        <description>Switch Statement with no case, ok</description>
         <rule-property name="minimumNumberCaseForASwitch">3</rule-property>
-        <expected-problems>1</expected-problems>
+        <expected-problems>0</expected-problems>
         <code><![CDATA[
 public class DumbSwitch {
     public void foo(int i) {
-        switch (i) { }
+        switch (i) { } // This is detected by EmptyControlStatement
     }
 }
         ]]></code>