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:17 UTC
svn commit: r1876474 -
/pdfbox/branches/issue45/fontbox/src/main/java/org/apache/fontbox/cmap/CMap.java
Author: tilman
Date: Tue Apr 14 04:36:17 2020
New Revision: 1876474
URL: http://svn.apache.org/viewvc?rev=1876474&view=rev
Log:
PDFBOX-4811: return what was read instead of 0 to reproduce Adobe Reader behavior
Modified:
pdfbox/branches/issue45/fontbox/src/main/java/org/apache/fontbox/cmap/CMap.java
Modified: pdfbox/branches/issue45/fontbox/src/main/java/org/apache/fontbox/cmap/CMap.java
URL: http://svn.apache.org/viewvc/pdfbox/branches/issue45/fontbox/src/main/java/org/apache/fontbox/cmap/CMap.java?rev=1876474&r1=1876473&r2=1876474&view=diff
==============================================================================
--- pdfbox/branches/issue45/fontbox/src/main/java/org/apache/fontbox/cmap/CMap.java (original)
+++ pdfbox/branches/issue45/fontbox/src/main/java/org/apache/fontbox/cmap/CMap.java Tue Apr 14 04:36:17 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
}
/**