Changeset 2088 in josm for trunk/src/org/openstreetmap/josm/gui/tagging/ac/AutoCompletionList.java

Timestamp:

09.09.2009 19:38:48 (3 years ago)

Author:

Gubaer

Message:

see #3393: loooong delay when using presets with a large osm file

File:

• trunk/src/org/openstreetmap/josm/gui/tagging/ac/AutoCompletionList.java (modified) (5 diffs)

trunk/src/org/openstreetmap/josm/gui/tagging/ac/AutoCompletionList.java

@@ -4 +4 @@
 import java.util.Collection;
 import java.util.Collections;
+import java.util.HashMap;
 import java.util.List;
+import java.util.Map;
 
 import javax.swing.JTable;
@@ -33 +35 @@
     /** the filter expression */
     private String filter = null;
+    /** map from value to priority */
+    private Map<String,AutoCompletionListItem> valutToItemMap;
 
     /**
@@ -40 +44 @@
         list = new ArrayList<AutoCompletionListItem>();
         filtered = new ArrayList<AutoCompletionListItem>();
+        valutToItemMap = new HashMap<String, AutoCompletionListItem>();
     }
 
@@ -148 +153 @@
 
     protected void appendOrUpdatePriority(AutoCompletionListItem toadd) {
-        AutoCompletionListItem item = lookup(toadd.getValue());
+        AutoCompletionListItem item = valutToItemMap.get(toadd.getValue());
         if (item == null) {
             // new item does not exist yet. Add it to the list
             //
             list.add(toadd);
+            valutToItemMap.put(toadd.getValue(), toadd);
         } else {
             // new item already exists. Update priority if necessary
@@ -191 +197 @@
         return false;
     }
-
-    /**
-     * 
-     * @param value a specific value
-     * @return  the auto completion item for this value; null, if there is no
-     *   such auto completion item
-     */
-    public AutoCompletionListItem lookup(String value) {
-        if (value == null)
-            return null;
-        for (AutoCompletionListItem item : list) {
-            if (item.getValue().equals(value))
-                return item;
-        }
-        return null;
-    }
-
 
     /**