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/13 20:31:04 UTC

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

Author: jahewson
Date: Tue Jan 13 19:31:04 2015
New Revision: 1651448

URL: http://svn.apache.org/r1651448
Log:
PDFBOX-2598: Fix backwards check for built-in 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=1651448&r1=1651447&r2=1651448&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 Tue Jan 13 19:31:04 2015
@@ -383,15 +383,15 @@ public class CFFParser
         if (charsetEntry != null)
         {
             int charsetId = charsetEntry.getNumber(0).intValue();
-            if (isCIDFont && charsetId == 0)
+            if (!isCIDFont && charsetId == 0)
             {
                 charset = CFFISOAdobeCharset.getInstance();
             }
-            else if (isCIDFont && charsetId == 1)
+            else if (!isCIDFont && charsetId == 1)
             {
                 charset = CFFExpertCharset.getInstance();
             }
-            else if (isCIDFont && charsetId == 2)
+            else if (!isCIDFont && charsetId == 2)
             {
                 charset = CFFExpertSubsetCharset.getInstance();
             }