You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pdfbox.apache.org by le...@apache.org on 2009/02/10 07:31:17 UTC

svn commit: r742864 - /incubator/pdfbox/trunk/src/main/java/org/apache/pdfbox/pdmodel/PDResources.java

Author: lehmi
Date: Tue Feb 10 06:31:13 2009
New Revision: 742864

URL: http://svn.apache.org/viewvc?rev=742864&view=rev
Log:
PDFBOX-421: avoid NPE because of a missing font-cache

Modified:
    incubator/pdfbox/trunk/src/main/java/org/apache/pdfbox/pdmodel/PDResources.java

Modified: incubator/pdfbox/trunk/src/main/java/org/apache/pdfbox/pdmodel/PDResources.java
URL: http://svn.apache.org/viewvc/incubator/pdfbox/trunk/src/main/java/org/apache/pdfbox/pdmodel/PDResources.java?rev=742864&r1=742863&r2=742864&view=diff
==============================================================================
--- incubator/pdfbox/trunk/src/main/java/org/apache/pdfbox/pdmodel/PDResources.java (original)
+++ incubator/pdfbox/trunk/src/main/java/org/apache/pdfbox/pdmodel/PDResources.java Tue Feb 10 06:31:13 2009
@@ -121,13 +121,17 @@
             {
                 COSDictionary fontDictionary = (COSDictionary)font;
                 PDFont newFont = null;
-                if (fontCache.containsKey(fontName.getName()))
-                	newFont = (PDFont)fontCache.get(fontName.getName());
-                else {
-                	newFont = PDFontFactory.createFont( fontDictionary );
-                	fontCache.put(fontName.getName(), newFont);
+                if (fontCache != null) {
+	                if (fontCache.containsKey(fontName.getName()))
+	                	newFont = (PDFont)fontCache.get(fontName.getName());
+	                else {
+	                	newFont = PDFontFactory.createFont( fontDictionary );
+	                	fontCache.put(fontName.getName(), newFont);
+	                }
                 }
-                actuals.put( fontName.getName(), newFont);
+                else
+                	newFont = PDFontFactory.createFont( fontDictionary );
+	            actuals.put( fontName.getName(), newFont);
             }
         }
         return retval;