From 7cd40f72ae9ede04d4f3bc4c95057f6e68d4089e Mon Sep 17 00:00:00 2001
From: Olivier Lamy <olamy@apache.org>
Date: Thu, 21 Mar 2024 21:44:12 +1000
Subject: [PATCH] remove useless spotbugs checks

Signed-off-by: Olivier Lamy <olamy@apache.org>
---
 pom.xml | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/pom.xml b/pom.xml
index 7559b5cb605..2242bd9d3ac 100644
--- a/pom.xml
+++ b/pom.xml
@@ -2249,6 +2249,14 @@
         <!-- Defines a SpotBugs effort. Use "Max" to maximize the scan depth -->
         <spotbugs.effort>Default</spotbugs.effort>
         <spotbugs.failOnError>false</spotbugs.failOnError>
+        <!--
+          The MS_EXPOSE_REP, EI_EXPOSE_REP, EI_EXPOSE_REP2, EI_EXPOSE_STATIC_REP2, MS_EXPOSE_BUF,
+          EI_EXPOSE_BUF, EI_EXPOSE_STATIC_BUF2, and EI_EXPOSE_BUF2 bug patterns are noisy and create
+          little value; therefore, we suppress them globally.
+
+          The CT_CONSTRUCTOR_THROW bug pattern is related to libraries used with SecurityManage.
+        -->
+        <spotbugs.omitVisitors>FindReturnRef,ConstructorThrow</spotbugs.omitVisitors>
         <spotbugs.onlyAnalyze>org.eclipse.jetty.*</spotbugs.onlyAnalyze>
         <spotbugs.skip>false</spotbugs.skip>
         <!-- Defines a SpotBugs threshold. Use "Low" to discover low-priority bugs -->