This is an automated email from the ASF dual-hosted git repository.
pascalschumacher pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-lang.git
commit c9ca06e496dee2b22a3753652735c0bca60afc7e
Author: pascalschumacher <pascalschumac...@gmx.net>
AuthorDate: Fri Dec 20 13:28:20 2019 +0100
Remove no longer necessary SpotBugs excludes.
---
spotbugs-exclude-filter.xml | 10 ----------
1 file changed, 10 deletions(-)
diff --git a/spotbugs-exclude-filter.xml b/spotbugs-exclude-filter.xml
index 2dd4c26..f24aa1d 100644
--- a/spotbugs-exclude-filter.xml
+++ b/spotbugs-exclude-filter.xml
@@ -100,11 +100,6 @@
<Bug pattern="CN_IDIOM_NO_SUPER_CALL" />
</Match>
- <!-- Reason: Testing shows that new Integer(...) etc is quicker than
Integer.valueOf -->
- <Match>
- <Bug pattern="DM_NUMBER_CTOR" />
- </Match>
-
<!-- Reason: FindBugs 2.0.2 used in maven-findbugs-plugin 2.5.2 seems to
have problems with detection of default cases
in switch statements. All the excluded methods have switch statements that
conatin a default case. -->
<Match>
@@ -112,11 +107,6 @@
<Method name="createNumber" />
<Bug pattern="SF_SWITCH_NO_DEFAULT" />
</Match>
- <Match>
- <Class name="org.apache.commons.lang3.text.ExtendedMessageFormat"/>
- <Method name="insertFormats" />
- <Bug pattern="SF_SWITCH_NO_DEFAULT" />
- </Match>
<!-- Reason: FindBugs does not correctly recognize default branches in
switch statements without break statements.
See, e.g., the report at https://sourceforge.net/p/findbugs/bugs/1298 -->
<Match>
