Merge branch 'master' into 3.0.X

apache · Jul 29, 2024 · 06e91ba · 06e91ba
2 parents 505071a + 8f0dc74
commit 06e91ba
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 69 deletions.
diff --git a/datasketches-memory-java8/src/main/java/org/apache/datasketches/memory/MurmurHash3v2.java b/datasketches-memory-java8/src/main/java/org/apache/datasketches/memory/MurmurHash3v2.java
@@ -299,7 +299,7 @@ public static long[] hash(
           k1 ^= (unsafe.getByte(uObj, cumOff) & 0xFFL);
           break;
         }
-        //default: break; //can't happen
+        default: break; //can't happen
       }
 
       h1 ^= mixK1(k1);

diff --git a/...ava8/src/main/java/org/apache/datasketches/memory/internal/AllocateDirectWritableMap.java b/...ava8/src/main/java/org/apache/datasketches/memory/internal/AllocateDirectWritableMap.java
@@ -50,7 +50,7 @@
  * @author Praveenkumar Venkatesan
  */
 @SuppressWarnings("restriction")
-class AllocateDirectWritableMap {
+final class AllocateDirectWritableMap {
   static final Logger LOG = Logger.getLogger(AllocateDirectWritableMap.class.getCanonicalName());
 
   private static final int MAP_RO = 0;

diff --git a/tools/FindBugsExcludeFilter.xml b/tools/FindBugsExcludeFilter.xml
@@ -21,76 +21,13 @@ under the License.
 
 <FindBugsFilter> <!-- memory -->
 
-  <!-- Too many false positives to be useful.  I could not make it happy :( -->
+  <!-- Ignore all test code -->
   <Match>
-    <Bug pattern="SF_SWITCH_NO_DEFAULT" />
-  </Match>
-
-  <Match>   <!-- Exclude for test classes; too many False Positives. -->
-    <Bug pattern="NP_NULL_PARAM_DEREF_NONVIRTUAL" />
     <Class name="~.*\.*Test" />
   </Match>
-
-  <Match>   <!-- Exclude for test classes; too many False Positives. -->
-    <Bug pattern="DLS_DEAD_LOCAL_STORE" />
-    <Class name="~.*\.*Test" />
+
+  <Match> <!-- harmless, not going to fix right now -->
+    <Bug pattern="INT_VACUOUS_BIT_OPERATION" />
   </Match>
 
 </FindBugsFilter>
-
-
-
-<!--  Examples: -->
-
-<!-- Exclude java.* classes -->
-  <!--
-  <Match>
-    <Package name="java\.*" />
-  </Match>
--->
-
-  <!-- Exclude test classes -->
-<!-- 
-  <Match>
-    <Class name="~.*\.*Test" />
-  </Match>
--->
-
-<!--
-     <Match>
-       <Class name="com.foobar.ClassNotToBeAnalyzed" />
-     </Match>
--->
-<!--
-     <Match>
-       <Class name="com.foobar.ClassWithSomeBugsMatched" />
-       <Bug code="DE,UrF,SIC" />
-     </Match>
--->
-     <!-- Match all XYZ violations. -->
-<!--
-     <Match>
-       <Bug code="XYZ" />
-     </Match>
--->
-     <!-- Match all doublecheck violations in these methods of "AnotherClass". -->
-<!--
-     <Match>
-       <Class name="com.foobar.AnotherClass" />
-       <Or>
-         <Method name="nonOverloadedMethod" />
-         <Method name="frob" params="int,java.lang.String" returns="void" />
-         <Method name="blat" params="" returns="boolean" />
-       </Or>
-       <Bug code="DC" />
-     </Match>
--->
-     <!-- A method with a dead local store false positive (medium priority). -->
-<!--
-     <Match>
-       <Class name="com.foobar.MyClass" />
-       <Method name="someMethod" />
-       <Bug pattern="DLS_DEAD_LOCAL_STORE" />
-       <Priority value="2" />
-     </Match>
--->