From e4e3dfff33ec37977f466fae2643996793a5e127 Mon Sep 17 00:00:00 2001
From: Sebb <sebb@apache.org>
Date: Sun, 17 Dec 2023 00:04:43 +0000
Subject: [PATCH 1/3] Avoid a spotbugs complaint [skip ci]

---
 .../java/org/apache/bcel/classfile/ConstantUtf8.java     | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/src/main/java/org/apache/bcel/classfile/ConstantUtf8.java b/src/main/java/org/apache/bcel/classfile/ConstantUtf8.java
index 51a843c861..edbdfef8b4 100644
--- a/src/main/java/org/apache/bcel/classfile/ConstantUtf8.java
+++ b/src/main/java/org/apache/bcel/classfile/ConstantUtf8.java
@@ -178,6 +178,11 @@ static void printStats() {
             Cache.MAX_ENTRY_SIZE);
     }
 
+    // Avoid Spotbugs complaint about Write to static field
+    private static void countCreated() {
+        created++;
+    }
+
     private final String value;
 
     /**
@@ -198,7 +203,7 @@ public ConstantUtf8(final ConstantUtf8 constantUtf8) {
     ConstantUtf8(final DataInput dataInput) throws IOException {
         super(Const.CONSTANT_Utf8);
         value = dataInput.readUTF();
-        created++;
+        countCreated();
     }
 
     /**
@@ -207,7 +212,7 @@ public ConstantUtf8(final ConstantUtf8 constantUtf8) {
     public ConstantUtf8(final String value) {
         super(Const.CONSTANT_Utf8);
         this.value = Objects.requireNonNull(value, "value");
-        created++;
+        countCreated();
     }
 
     /**

From faba736fb7ff57c422e19726aba63a31c311b2b7 Mon Sep 17 00:00:00 2001
From: Sebb <sebb@apache.org>
Date: Sun, 17 Dec 2023 00:05:16 +0000
Subject: [PATCH 2/3] Avoid a spotbugs complaint [skip ci]

---
 src/main/java/org/apache/bcel/generic/InstructionList.java | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/src/main/java/org/apache/bcel/generic/InstructionList.java b/src/main/java/org/apache/bcel/generic/InstructionList.java
index c373e3b3f5..ca053e4418 100644
--- a/src/main/java/org/apache/bcel/generic/InstructionList.java
+++ b/src/main/java/org/apache/bcel/generic/InstructionList.java
@@ -1129,6 +1129,9 @@ public void setPositions(final boolean check) { // called by code in other packa
             case Const.LOOKUPSWITCH:
                 maxAdditionalBytes += 3;
                 break;
+            default:
+                // TODO should this be an error?
+                break;
             }
             index += i.getLength();
         }

From bb9c610e5e2dffd641b19681a480ded1f4025117 Mon Sep 17 00:00:00 2001
From: Sebb <sebb@apache.org>
Date: Sun, 17 Dec 2023 00:06:19 +0000
Subject: [PATCH 3/3] Ignore specific spotbugs errors not the number

---
 pom.xml                              |  2 -
 src/conf/spotbugs-exclude-filter.xml | 69 ++++++++++++++++++++++++++--
 2 files changed, 66 insertions(+), 5 deletions(-)

diff --git a/pom.xml b/pom.xml
index 833a7f1747..2035b710b3 100644
--- a/pom.xml
+++ b/pom.xml
@@ -330,8 +330,6 @@
           <threshold>Normal</threshold>
           <effort>Default</effort>
           <excludeFilterFile>src/conf/spotbugs-exclude-filter.xml</excludeFilterFile>
-          <!-- Hacky: Don't allow new SpotBugs errors --> 
-          <maxAllowedViolations>9</maxAllowedViolations>
         </configuration>
       </plugin>
       <plugin>
diff --git a/src/conf/spotbugs-exclude-filter.xml b/src/conf/spotbugs-exclude-filter.xml
index 0698d4fa45..f917b60c7f 100644
--- a/src/conf/spotbugs-exclude-filter.xml
+++ b/src/conf/spotbugs-exclude-filter.xml
@@ -62,10 +62,29 @@
 
   <!-- Binary compatibility -->
   <Match>
-    <Bug pattern="EI_EXPOSE_STATIC_REP2"/>
-    <Method name="clone"/>
-    <Class name="org\.apache\.bcel\.Repository"/>
+    <Class name="org.apache.bcel.Repository"/>
     <Method name="setRepository"/>
+    <Bug pattern="EI_EXPOSE_STATIC_REP2"/>
+  </Match>
+  
+  <Match>
+    <Class name="org.apache.bcel.Repository"/>
+    <Method name="getRepository"/>
+    <Bug pattern="MS_EXPOSE_REP"/>
+  </Match>
+
+
+  <!--
+    TODO: this should probably be sinplified
+     Medium: Complicated, subtle or wrong increment in for-loop
+      org.apache.bcel.util.BCELifier.printFlags(int, BCELifier$FLAGS)
+      [org.apache.bcel.util.BCELifier] At BCELifier.java:[line 118]
+      QF_QUESTIONABLE_FOR_LOOP
+   -->
+  <Match>
+    <Class name="org.apache.bcel.util.BCELifier"/>
+    <Method name="printFlags" params="int, org.apache.bcel.util.BCELifier$FLAGS" returns="java.lang.String"/>
+    <Bug pattern="QF_QUESTIONABLE_FOR_LOOP"/>
   </Match>
 
   <!-- Reason: TODO, perhaps? -->
@@ -91,4 +110,48 @@
     </Or>
   </Match>
 
+  <!--
+    This is intentional
+    Error:  Switch statement found in org.apache.bcel.util.BCELFactory.visitAllocationInstruction(AllocationInstruction)
+     where one case falls through to the next case
+     [org.apache.bcel.util.BCELFactory, org.apache.bcel.util.BCELFactory]
+     At BCELFactory.java:[lines 188-191]
+     Another occurrence at BCELFactory.java:[lines 192-196] SF_SWITCH_FALLTHROUGH
+  -->
+  <Match>
+    <Class name="org.apache.bcel.util.BCELFactory"/>
+    <Method name="visitAllocationInstruction"/>
+    <Bug pattern="SF_SWITCH_FALLTHROUGH"/>
+  </Match>
+
+  <!--
+    Class is deprecated
+    Error:  The class name org.apache.bcel.util.ClassLoader
+      shadows the simple name of the superclass java.lang.ClassLoader
+      [org.apache.bcel.util.ClassLoader] At ClassLoader.java:[lines 59-178]
+      NM_SAME_SIMPLE_NAME_AS_SUPERCLASS
+    Error:  org.apache.bcel.util.ClassLoader.DEFAULT_IGNORED_PACKAGES should be package protected
+    [org.apache.bcel.util.ClassLoader] At ClassLoader.java:[line 59]
+    MS_PKGPROTECT
+  -->
+  <Match>
+    <Class name="org.apache.bcel.util.ClassLoader"/>
+    <Or>
+      <Bug pattern="NM_SAME_SIMPLE_NAME_AS_SUPERCLASS"/>
+      <Bug pattern="MS_PKGPROTECT"/>
+    </Or>
+  </Match>
+
+  <!--
+    TODO: field is deprecated in preparation for making it private later
+    Error:  org.apache.bcel.verifier.structurals.Frame._this should be package protected
+    [org.apache.bcel.verifier.structurals.Frame] In Frame.java
+      MS_PKGPROTECT
+  -->
+  <Match>
+    <Class name="org.apache.bcel.verifier.structurals.Frame"/>
+    <Field name="_this"/>
+    <Bug pattern="MS_PKGPROTECT"/>
+  </Match>
+
 </FindBugsFilter>