This is an automated email from the ASF dual-hosted git repository.

jleroux pushed a commit to branch trunk
in repository https://gitbox.apache.org/repos/asf/ofbiz-framework.git


The following commit(s) were added to refs/heads/trunk by this push:
     new 8588de5  Improved: Fix some bugs Spotbugs reports (OFBIZ-12386)
8588de5 is described below

commit 8588de579c5e8a326f0845a212fe06bdcd65a1c0
Author: Jacques Le Roux <jacques.le.r...@les7arts.com>
AuthorDate: Sun Dec 26 14:02:17 2021 +0100

    Improved: Fix some bugs Spotbugs reports (OFBIZ-12386)
    
    I prefer to remove this exclusion as it may depend of Spotbugs version.
---
 spotbugs/exclude.xml | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/spotbugs/exclude.xml b/spotbugs/exclude.xml
index bceb461..4776f53 100644
--- a/spotbugs/exclude.xml
+++ b/spotbugs/exclude.xml
@@ -155,11 +155,5 @@
     <Field name="baseUrl" />
     <Bug pattern="ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD" />
   </Match>
-  <Match>
-    <!-- not a problem, it's fixed, but despite SpotBugs continues to report 
the same error -->
-    <Class 
name="org.apache.ofbiz.base.util.collections.FlexibleServletAccessor" />
-    <Method name="equals" />
-    <Bug pattern="NP_EQUALS_SHOULD_HANDLE_NULL_ARGUMENT" />
-  </Match>  
 <!-- ^ ^ not a problem, false positives -->
 </FindBugsFilter>

Reply via email to