You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pdfbox.apache.org by ja...@apache.org on 2015/01/23 19:59:09 UTC

svn commit: r1654311 - /pdfbox/trunk/xmpbox/src/main/java/org/apache/xmpbox/xml/XmpSerializer.java

Author: jahewson
Date: Fri Jan 23 18:59:08 2015
New Revision: 1654311

URL: http://svn.apache.org/r1654311
Log:
PDFBOX-2628: XmpSerializer() will never throw XmpSerializationException

Modified:
    pdfbox/trunk/xmpbox/src/main/java/org/apache/xmpbox/xml/XmpSerializer.java

Modified: pdfbox/trunk/xmpbox/src/main/java/org/apache/xmpbox/xml/XmpSerializer.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/xmpbox/src/main/java/org/apache/xmpbox/xml/XmpSerializer.java?rev=1654311&r1=1654310&r2=1654311&view=diff
==============================================================================
--- pdfbox/trunk/xmpbox/src/main/java/org/apache/xmpbox/xml/XmpSerializer.java (original)
+++ pdfbox/trunk/xmpbox/src/main/java/org/apache/xmpbox/xml/XmpSerializer.java Fri Jan 23 18:59:08 2015
@@ -57,7 +57,7 @@ public class XmpSerializer
 
     private boolean parseTypeResourceForLi = true;
 
-    public XmpSerializer() throws XmpSerializationException
+    public XmpSerializer()
     {
         // xml init
         DocumentBuilderFactory builderFactory = DocumentBuilderFactory.newInstance();
@@ -67,9 +67,9 @@ public class XmpSerializer
         }
         catch (ParserConfigurationException e)
         {
-            throw new XmpSerializationException("Failed to init XmpSerializer", e);
+            // never happens, because we don't call builderFactory#setAttribute
+            throw new RuntimeException(e);
         }
-
     }
 
     public void serialize(XMPMetadata metadata, OutputStream os, boolean withXpacket) throws TransformerException