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 2023/03/11 11:32:42 UTC
svn commit: r1908277 - /pdfbox/branches/2.0/fontbox/src/main/java/org/apache/fontbox/ttf/CmapSubtable.java
Author: tilman
Date: Sat Mar 11 11:32:42 2023
New Revision: 1908277
URL: http://svn.apache.org/viewvc?rev=1908277&view=rev
Log:
PDFBOX-4892: optimize, as suggested by valerybokov
Modified:
pdfbox/branches/2.0/fontbox/src/main/java/org/apache/fontbox/ttf/CmapSubtable.java
Modified: pdfbox/branches/2.0/fontbox/src/main/java/org/apache/fontbox/ttf/CmapSubtable.java
URL: http://svn.apache.org/viewvc/pdfbox/branches/2.0/fontbox/src/main/java/org/apache/fontbox/ttf/CmapSubtable.java?rev=1908277&r1=1908276&r2=1908277&view=diff
==============================================================================
--- pdfbox/branches/2.0/fontbox/src/main/java/org/apache/fontbox/ttf/CmapSubtable.java (original)
+++ pdfbox/branches/2.0/fontbox/src/main/java/org/apache/fontbox/ttf/CmapSubtable.java Sat Mar 11 11:32:42 2023
@@ -45,7 +45,7 @@ public class CmapSubtable implements Cma
private long subTableOffset;
private int[] glyphIdToCharacterCode;
private final Map<Integer, List<Integer>> glyphIdToCharacterCodeMultiple = new HashMap<Integer, List<Integer>>();
- private Map<Integer, Integer> characterCodeToGlyphId= new HashMap<Integer, Integer>();
+ private Map<Integer, Integer> characterCodeToGlyphId = Collections.emptyMap();
/**
* This will read the required data from the stream.
@@ -471,7 +471,7 @@ public class CmapSubtable implements Cma
List<Integer> mappedValues = glyphIdToCharacterCodeMultiple.get(entry.getValue());
if (mappedValues == null)
{
- mappedValues = new ArrayList<Integer>();
+ mappedValues = new ArrayList<Integer>(2);
glyphIdToCharacterCodeMultiple.put(entry.getValue(), mappedValues);
mappedValues.add(glyphIdToCharacterCode[entry.getValue()]);
// mark value as multiple mapping