You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pdfbox.apache.org by ti...@apache.org on 2015/01/14 21:56:55 UTC

svn commit: r1651820 - in /pdfbox/trunk/preflight/src/main/java/org/apache/pdfbox/preflight/font: container/FontContainer.java util/GlyphException.java

Author: tilman
Date: Wed Jan 14 20:56:55 2015
New Revision: 1651820

URL: http://svn.apache.org/r1651820
Log:
PDFBOX-2576: pass exception in constructor of exception; make fields final

Modified:
    pdfbox/trunk/preflight/src/main/java/org/apache/pdfbox/preflight/font/container/FontContainer.java
    pdfbox/trunk/preflight/src/main/java/org/apache/pdfbox/preflight/font/util/GlyphException.java

Modified: pdfbox/trunk/preflight/src/main/java/org/apache/pdfbox/preflight/font/container/FontContainer.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/preflight/src/main/java/org/apache/pdfbox/preflight/font/container/FontContainer.java?rev=1651820&r1=1651819&r2=1651820&view=diff
==============================================================================
--- pdfbox/trunk/preflight/src/main/java/org/apache/pdfbox/preflight/font/container/FontContainer.java (original)
+++ pdfbox/trunk/preflight/src/main/java/org/apache/pdfbox/preflight/font/container/FontContainer.java Wed Jan 14 20:56:55 2015
@@ -133,7 +133,7 @@ public abstract class FontContainer<T ex
         {
             throw new GlyphException(PreflightConstants.ERROR_FONTS_GLYPH, code,
                     "Unexpected error during the width validation for the character code " + code +
-                    " : " + e.getMessage());
+                    " : " + e.getMessage(), e);
         }
     }
 

Modified: pdfbox/trunk/preflight/src/main/java/org/apache/pdfbox/preflight/font/util/GlyphException.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/preflight/src/main/java/org/apache/pdfbox/preflight/font/util/GlyphException.java?rev=1651820&r1=1651819&r2=1651820&view=diff
==============================================================================
--- pdfbox/trunk/preflight/src/main/java/org/apache/pdfbox/preflight/font/util/GlyphException.java (original)
+++ pdfbox/trunk/preflight/src/main/java/org/apache/pdfbox/preflight/font/util/GlyphException.java Wed Jan 14 20:56:55 2015
@@ -23,8 +23,8 @@ package org.apache.pdfbox.preflight.font
 
 public class GlyphException extends Exception
 {
-    private String errorCode;
-    private int invalidCid;
+    private final String errorCode;
+    private final int invalidCid;
 
     public GlyphException(String errorCode, int cid)
     {
@@ -39,6 +39,13 @@ public class GlyphException extends Exce
         this.errorCode = errorCode;
         this.invalidCid = cid;
     }
+
+    public GlyphException(String errorCode, int cid, String message, Throwable cause)
+    {
+        super(message, cause);
+        this.errorCode = errorCode;
+        this.invalidCid = cid;
+    }
 
     public String getErrorCode()
     {