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 2014/12/18 18:56:20 UTC

svn commit: r1646500 - /pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/cff/CFFParser.java

Author: tilman
Date: Thu Dec 18 17:56:19 2014
New Revision: 1646500

URL: http://svn.apache.org/r1646500
Log:
PDFBOX-2571: get the pre-rev-1646091 behavior for non CIDFonts that don't have a charset

Modified:
    pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/cff/CFFParser.java

Modified: pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/cff/CFFParser.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/cff/CFFParser.java?rev=1646500&r1=1646499&r2=1646500&view=diff
==============================================================================
--- pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/cff/CFFParser.java (original)
+++ pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/cff/CFFParser.java Thu Dec 18 17:56:19 2014
@@ -403,8 +403,16 @@ public class CFFParser
         }
         else
         {
-            // a CID font with no charset does not default to any predefined charset
-            charset = new EmptyCharset();
+            if (isCIDFont)
+            {
+                // a CID font with no charset does not default to any predefined charset
+                charset = new EmptyCharset();
+            }
+            else
+            {
+                //FIXME PDFBOX-2571
+                charset = CFFISOAdobeCharset.getInstance();
+            }
         }
         font.setCharset(charset);