bug 61170: add comment why we should catch exceptions in SAXHelper

git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1798793 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
Javen O'Neal 2017-06-15 05:08:24 +00:00
parent 7413857e16
commit 285ac85870
1 changed files with 4 additions and 0 deletions

View File

@ -73,6 +73,10 @@ public final class SAXHelper {
logger.log(POILogger.WARN, "Failed to create SAXParserFactory", e); logger.log(POILogger.WARN, "Failed to create SAXParserFactory", e);
throw new RuntimeException("Failed to create SAXParserFactory", e); throw new RuntimeException("Failed to create SAXParserFactory", e);
} catch (Error e) { } catch (Error e) {
// catches NoClassDefFoundError, which may be due to a local class path issue
// This may occur if the code is run inside a web container
// or a restricted JVM
// See bug 61170: https://bz.apache.org/bugzilla/show_bug.cgi?id=61170
logger.log(POILogger.WARN, "Failed to create SAXParserFactory", e); logger.log(POILogger.WARN, "Failed to create SAXParserFactory", e);
throw e; throw e;
} }