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 2021/04/14 18:49:59 UTC
svn commit: r1888768 -
/pdfbox/branches/2.0/fontbox/src/main/java/org/apache/fontbox/cff/CFFParser.java
Author: tilman
Date: Wed Apr 14 18:49:59 2021
New Revision: 1888768
URL: http://svn.apache.org/viewvc?rev=1888768&view=rev
Log:
PDFBOX-4892: performance improvement, as suggested by valerybokov; comment from CFF specification
Modified:
pdfbox/branches/2.0/fontbox/src/main/java/org/apache/fontbox/cff/CFFParser.java
Modified: pdfbox/branches/2.0/fontbox/src/main/java/org/apache/fontbox/cff/CFFParser.java
URL: http://svn.apache.org/viewvc/pdfbox/branches/2.0/fontbox/src/main/java/org/apache/fontbox/cff/CFFParser.java?rev=1888768&r1=1888767&r2=1888768&view=diff
==============================================================================
--- pdfbox/branches/2.0/fontbox/src/main/java/org/apache/fontbox/cff/CFFParser.java (original)
+++ pdfbox/branches/2.0/fontbox/src/main/java/org/apache/fontbox/cff/CFFParser.java Wed Apr 14 18:49:59 2021
@@ -822,9 +822,9 @@ public class CFFParser
int gid = 1;
for (int i = 0; i < encoding.nRanges; i++)
{
- int rangeFirst = dataInput.readCard8();
- int rangeLeft = dataInput.readCard8();
- for (int j = 0; j < 1 + rangeLeft; j++)
+ int rangeFirst = dataInput.readCard8(); // First code in range
+ int rangeLeft = dataInput.readCard8(); // Codes left in range (excluding first)
+ for (int j = 0; j <= rangeLeft; j++)
{
int sid = charset.getSIDForGID(gid);
int code = rangeFirst + j;