Author: tilman
Date: Sun Sep 11 18:20:58 2022
New Revision: 1904001

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

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

Modified: 
pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/cos/COSDictionary.java
URL: 
http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/cos/COSDictionary.java?rev=1904001&r1=1904000&r2=1904001&view=diff
==============================================================================
--- pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/cos/COSDictionary.java 
(original)
+++ pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/cos/COSDictionary.java 
Sun Sep 11 18:20:58 2022
@@ -74,11 +74,7 @@ public class COSDictionary extends COSBa
     public COSDictionary(COSDictionary dict)
     {
         updateState = new COSUpdateState(this);
-        if (items instanceof SmallMap && items.size() + dict.items.size() >= 
100)
-        {
-            items = new LinkedHashMap<>(items);
-        }
-        items.putAll(dict.items);
+        addAll(dict);
     }
 
     /**
@@ -1271,11 +1267,15 @@ public class COSDictionary extends COSBa
      * This will add all of the dictionaries keys/values to this dictionary. 
Existing key/value pairs will be
      * overwritten.
      *
-     * @param dic The dictionaries to get the key/value pairs from.
+     * @param dict The dictionaries to get the key/value pairs from.
      */
-    public void addAll(COSDictionary dic)
+    public void addAll(COSDictionary dict)
     {
-        items.putAll(dic.items);
+        if (items instanceof SmallMap && items.size() + dict.items.size() >= 
100)
+        {
+            items = new LinkedHashMap<>(items);
+        }
+        items.putAll(dict.items);
     }
 
     /**


Reply via email to