[java-opensaml COMMIT] in /trunk/opensaml-util/src/main/java/org/opensaml/util/xml: BasicParserPool.java ParserPool.java

noreply at shibboleth.net noreply at shibboleth.net
Sun Oct 2 14:38:57 BST 2011


Author: lajoie
Date: Sun Oct  2 14:38:56 2011
New Revision: 2913

URL: http://svn.shibboleth.net/view/java-opensaml?rev=2913&view=rev
Log:
tweak some logging messages

Modified:
    trunk/opensaml-util/src/main/java/org/opensaml/util/xml/BasicParserPool.java
    trunk/opensaml-util/src/main/java/org/opensaml/util/xml/ParserPool.java

Modified: trunk/opensaml-util/src/main/java/org/opensaml/util/xml/BasicParserPool.java
URL: http://svn.shibboleth.net/view/java-opensaml/trunk/opensaml-util/src/main/java/org/opensaml/util/xml/BasicParserPool.java?rev=2913&r1=2912&r2=2913&view=diff
==============================================================================
--- trunk/opensaml-util/src/main/java/org/opensaml/util/xml/BasicParserPool.java (original)
+++ trunk/opensaml-util/src/main/java/org/opensaml/util/xml/BasicParserPool.java Sun Oct  2 14:38:56 2011
@@ -227,9 +227,9 @@
             final Document document = builder.parse(input);
             return document;
         } catch (SAXException e) {
-            throw new XMLParserException("Invalid XML", e);
+            throw new XMLParserException("Unable to parse inputstream, it containsed invalid XML", e);
         } catch (IOException e) {
-            throw new XMLParserException("Unable to read XML from input stream", e);
+            throw new XMLParserException("Unable to read data from input stream", e);
         } finally {
             returnBuilder(builder);
         }
@@ -532,7 +532,7 @@
 
             return builder;
         } catch (ParserConfigurationException e) {
-            log.error("Unable to create new document builder", e);
+            log.debug("Unable to create new document builder", e);
             throw new XMLParserException("Unable to create new document builder", e);
         }
     }

Modified: trunk/opensaml-util/src/main/java/org/opensaml/util/xml/ParserPool.java
URL: http://svn.shibboleth.net/view/java-opensaml/trunk/opensaml-util/src/main/java/org/opensaml/util/xml/ParserPool.java?rev=2913&r1=2912&r2=2913&view=diff
==============================================================================
--- trunk/opensaml-util/src/main/java/org/opensaml/util/xml/ParserPool.java (original)
+++ trunk/opensaml-util/src/main/java/org/opensaml/util/xml/ParserPool.java Sun Oct  2 14:38:56 2011
@@ -25,9 +25,10 @@
 
 import org.w3c.dom.Document;
 
-/**
- * A pool of XML parsers.
- */
+//TODO needs to implement destructable and verifiable component
+//TODO may need to implement initializable component
+
+/** A pool of XML parsers. */
 public interface ParserPool {
 
     /**



More information about the commits mailing list