From: Dominik Stadler <centic@apache.org>
Date: Thu, 24 Feb 2022 18:12:25 +0000 (+0000)
Subject: Fix some SpotBugs reports
X-Git-Tag: REL_5_2_1~8
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=25769795de779a2a90829a61bfc74196b43abdc2;p=poi.git

Fix some SpotBugs reports

git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1898383 13f79535-47bb-0310-9956-ffa450edef68
---

diff --git a/poi-integration/src/test/java/org/apache/poi/stress/AbstractFileHandler.java b/poi-integration/src/test/java/org/apache/poi/stress/AbstractFileHandler.java
index fec396281d..4bc11d0c40 100644
--- a/poi-integration/src/test/java/org/apache/poi/stress/AbstractFileHandler.java
+++ b/poi-integration/src/test/java/org/apache/poi/stress/AbstractFileHandler.java
@@ -44,7 +44,8 @@ import org.apache.poi.util.IOUtils;
  * in the integration tests, mostly text-extraction related at the moment.
  */
 public abstract class AbstractFileHandler implements FileHandler {
-    public static final Set<String> EXPECTED_EXTRACTOR_FAILURES = new HashSet<>(Arrays.asList(
+    // some FileHandlers extend this list!?!
+    protected static final Set<String> EXPECTED_EXTRACTOR_FAILURES = new HashSet<>(Arrays.asList(
         // password protected files without password
         // ... currently none ...
 
diff --git a/poi-integration/src/test/java/org/apache/poi/stress/FileHandlerKnown.java b/poi-integration/src/test/java/org/apache/poi/stress/FileHandlerKnown.java
index e7b1a8e7eb..c3f1e11137 100644
--- a/poi-integration/src/test/java/org/apache/poi/stress/FileHandlerKnown.java
+++ b/poi-integration/src/test/java/org/apache/poi/stress/FileHandlerKnown.java
@@ -18,7 +18,7 @@ package org.apache.poi.stress;
 
 import java.io.File;
 import java.io.InputStream;
-import java.util.function.Supplier;
+import java.lang.reflect.InvocationTargetException;
 
 @SuppressWarnings("unused")
 public enum FileHandlerKnown {
@@ -46,7 +46,8 @@ public enum FileHandlerKnown {
             // Because of no-scratchpad handling, we need to resort to reflection here
             String n = name().replace("NULL", "Null");
             return (FileHandler)Class.forName("org.apache.poi.stress." + n + "FileHandler").getDeclaredConstructor().newInstance();
-        } catch (Exception e) {
+        } catch (RuntimeException | ClassNotFoundException | NoSuchMethodException | InstantiationException |
+                IllegalAccessException | InvocationTargetException e) {
             return new NullFileHandler();
         }
     }