You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pdfbox.apache.org by le...@apache.org on 2021/09/04 11:11:50 UTC

svn commit: r1892896 - /pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/PDType1Font.java

Author: lehmi
Date: Sat Sep  4 11:11:50 2021
New Revision: 1892896

URL: http://svn.apache.org/viewvc?rev=1892896&view=rev
Log:
PDFBOX-5214: replace ConcurrentHashMap with HashMap as it is no longer needed for standard 14 fonts

Modified:
    pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/PDType1Font.java

Modified: pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/PDType1Font.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/PDType1Font.java?rev=1892896&r1=1892895&r2=1892896&view=diff
==============================================================================
--- pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/PDType1Font.java (original)
+++ pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/PDType1Font.java Sat Sep  4 11:11:50 2021
@@ -25,7 +25,6 @@ import java.util.Arrays;
 import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
-import java.util.concurrent.ConcurrentHashMap;
 import org.apache.commons.logging.Log;
 import org.apache.commons.logging.LogFactory;
 import org.apache.fontbox.EncodedFont;
@@ -93,7 +92,7 @@ public class PDType1Font extends PDSimpl
     /**
      * to improve encoding speed.
      */
-    private final Map <Integer,byte[]> codeToBytesMap;
+    private final Map<Integer, byte[]> codeToBytesMap = new HashMap<>();
     private Matrix fontMatrix;
     private BoundingBox fontBBox;
 
@@ -122,9 +121,6 @@ public class PDType1Font extends PDSimpl
             break;
         }
 
-        // standard 14 fonts may be accessed concurrently, as they are singletons
-        codeToBytesMap = new ConcurrentHashMap<>();
-
         // todo: could load the PFB font here if we wanted to support Standard 14 embedding
         type1font = null;
         FontMapping<FontBoxFont> mapping = FontMappers.instance()
@@ -181,7 +177,6 @@ public class PDType1Font extends PDSimpl
         isEmbedded = true;
         isDamaged = false;
         fontMatrixTransform = new AffineTransform();
-        codeToBytesMap = new HashMap<>();
     }
 
     /**
@@ -194,7 +189,6 @@ public class PDType1Font extends PDSimpl
     public PDType1Font(COSDictionary fontDictionary) throws IOException
     {
         super(fontDictionary);
-        codeToBytesMap = new HashMap<>();
 
         PDFontDescriptor fd = getFontDescriptor();
         Type1Font t1 = null;