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 2020/06/13 09:12:31 UTC

svn commit: r1878798 - /pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/Standard14Fonts.java

Author: tilman
Date: Sat Jun 13 09:12:30 2020
New Revision: 1878798

URL: http://svn.apache.org/viewvc?rev=1878798&view=rev
Log:
PDFBOX-4875: modify exception throwing sequence so that it is similar to 2.0

Modified:
    pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/Standard14Fonts.java

Modified: pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/Standard14Fonts.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/Standard14Fonts.java?rev=1878798&r1=1878797&r2=1878798&view=diff
==============================================================================
--- pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/Standard14Fonts.java (original)
+++ pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/Standard14Fonts.java Sat Jun 13 09:12:30 2020
@@ -109,9 +109,9 @@ final class Standard14Fonts
      * jar under /org/apache/pdfbox/resources/afm/
      *
      * @param fontName one of the standard 14 font names for which to lod the metrics.
-     * @throws IllegalArgumentException if no metrics exist for that font.
+     * @throws IOException if no metrics exist for that font.
      */
-    private static void loadMetrics(String fontName)
+    private static void loadMetrics(String fontName) throws IOException
     {
         String resourceName = "/org/apache/pdfbox/resources/afm/" + fontName + ".afm";
         try (InputStream resourceAsStream = PDType1Font.class.getResourceAsStream(resourceName);
@@ -121,10 +121,6 @@ final class Standard14Fonts
             FontMetrics metric = parser.parse(true);
             FONTS.put(fontName, metric);
         }
-        catch (IOException e)
-        {
-            throw new IllegalArgumentException(e);
-        }
     }
 
     /**
@@ -177,7 +173,14 @@ final class Standard14Fonts
             {
                 if (FONTS.get(baseName) == null)
                 {
-                    loadMetrics(baseName);
+                    try
+                    {
+                        loadMetrics(baseName);
+                    }
+                    catch (IOException e)
+                    {
+                        throw new IllegalArgumentException(e);
+                    }
                 }
             }
         }