bug 61170: add extra logging to SaxHelper.

Thanks to PJ Fanning for the patch!
This closes #58 on github.

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

diff --git a/src/ooxml/java/org/apache/poi/util/SAXHelper.java b/src/ooxml/java/org/apache/poi/util/SAXHelper.java
index 41f14fbcc28757f1c87d0a6463521d66779ce4fb..5149a222bb54c6bcb94c9b0fdeb5730439f03452 100644 (file)
--- a/src/ooxml/java/org/apache/poi/util/SAXHelper.java
+++ b/src/ooxml/java/org/apache/poi/util/SAXHelper.java
@@ -62,9 +62,20 @@ public final class SAXHelper {
     
     private static final SAXParserFactory saxFactory;
     static {
-        saxFactory = SAXParserFactory.newInstance();
-        saxFactory.setValidating(false);
-        saxFactory.setNamespaceAware(true);
+        try {
+            saxFactory = SAXParserFactory.newInstance();
+            saxFactory.setValidating(false);
+            saxFactory.setNamespaceAware(true);
+        } catch (RuntimeException re) {
+            logger.log(POILogger.WARN, "Failed to create SAXParserFactory", re);
+            throw re;
+        } catch (Exception e) {
+            logger.log(POILogger.WARN, "Failed to create SAXParserFactory", e);
+            throw new RuntimeException("Failed to create SAXParserFactory", e);
+        } catch (Error e) {
+            logger.log(POILogger.WARN, "Failed to create SAXParserFactory", e);
+            throw e;
+        }
     }
             
     private static void trySetSAXFeature(XMLReader xmlReader, String feature) {