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