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 2022/09/11 07:23:34 UTC

svn commit: r1903991 - /pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/cos/COSDictionary.java

Author: tilman
Date: Sun Sep 11 07:23:33 2022
New Revision: 1903991

URL: http://svn.apache.org/viewvc?rev=1903991&view=rev
Log:
PDFBOX-5499: use LinkedHashMap when SmallMap has more than 100 elements

Modified:
    pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/cos/COSDictionary.java

Modified: pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/cos/COSDictionary.java
URL: http://svn.apache.org/viewvc/pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/cos/COSDictionary.java?rev=1903991&r1=1903990&r2=1903991&view=diff
==============================================================================
--- pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/cos/COSDictionary.java (original)
+++ pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/cos/COSDictionary.java Sun Sep 11 07:23:33 2022
@@ -22,6 +22,7 @@ import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.Calendar;
 import java.util.Collection;
+import java.util.LinkedHashMap;
 import java.util.List;
 import java.util.Map;
 import java.util.Set;
@@ -62,6 +63,10 @@ public class COSDictionary extends COSBa
      */
     public COSDictionary(COSDictionary dict)
     {
+        if (items instanceof SmallMap && items.size() + dict.items.size() >= 100)
+        {
+            items = new LinkedHashMap<COSName, COSBase>(items);
+        }
         items.putAll(dict.items);
     }
 
@@ -214,6 +219,10 @@ public class COSDictionary extends COSBa
         }
         else
         {
+            if (items instanceof SmallMap && items.size() >= 100)
+            {
+                items = new LinkedHashMap<COSName, COSBase>(items);
+            }
             items.put(key, value);
         }
     }