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>