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/04/14 04:36:09 UTC

svn commit: r1876472 - /pdfbox/branches/2.0/fontbox/src/main/java/org/apache/fontbox/cmap/CMap.java

Author: tilman
Date: Tue Apr 14 04:36:09 2020
New Revision: 1876472

URL: http://svn.apache.org/viewvc?rev=1876472&view=rev
Log:
PDFBOX-4811: return what was read instead of 0 to reproduce Adobe Reader behavior

Modified:
    pdfbox/branches/2.0/fontbox/src/main/java/org/apache/fontbox/cmap/CMap.java

Modified: pdfbox/branches/2.0/fontbox/src/main/java/org/apache/fontbox/cmap/CMap.java
URL: http://svn.apache.org/viewvc/pdfbox/branches/2.0/fontbox/src/main/java/org/apache/fontbox/cmap/CMap.java?rev=1876472&r1=1876471&r2=1876472&view=diff
==============================================================================
--- pdfbox/branches/2.0/fontbox/src/main/java/org/apache/fontbox/cmap/CMap.java (original)
+++ pdfbox/branches/2.0/fontbox/src/main/java/org/apache/fontbox/cmap/CMap.java Tue Apr 14 04:36:09 2020
@@ -131,7 +131,7 @@ public class CMap
             seq += String.format("0x%02X (%04o) ", bytes[i], bytes[i]);
         }
         LOG.warn("Invalid character code sequence " + seq + "in CMap " + cmapName);
-        // PDFBOX-4811 reposition after first byte
+        // PDFBOX-4811 reposition to where we were after initial read
         if (in.markSupported())
         {
             in.reset();
@@ -141,7 +141,7 @@ public class CMap
             LOG.warn("mark() and reset() not supported, " + (maxCodeLength - 1) +
                      " bytes have been skipped");
         }
-        return 0;
+        return toInt(bytes, minCodeLength); // Adobe Reader behavior
     }
 
     /**