Changeset 2655 in josm for trunk/src/org/openstreetmap/josm/gui/dialogs/relation/RelationTreeModel.java

Timestamp:

2009-12-19T10:47:49+01:00 (14 years ago)

Author:

jttt

Message:

Fixed #4161 Major slowdown in recent versions, used correct pattern for listeners realized using CopyOnWriteArrayList

File:

• trunk/src/org/openstreetmap/josm/gui/dialogs/relation/RelationTreeModel.java (modified) (3 diffs)

trunk/src/org/openstreetmap/josm/gui/dialogs/relation/RelationTreeModel.java

@@ -4 +4 @@
 import java.util.List;
 import java.util.concurrent.CopyOnWriteArrayList;
-import java.util.logging.Logger;
 
 import javax.swing.event.TreeModelEvent;
@@ -24 +23 @@
  */
 public class RelationTreeModel implements TreeModel {
-    private static final Logger logger = Logger.getLogger(RelationTreeModel.class.getName());
+    //private static final Logger logger = Logger.getLogger(RelationTreeModel.class.getName());
 
     /** the root relation */
@@ -215 +214 @@
 
     public void addTreeModelListener(TreeModelListener l) {
-        synchronized (listeners) {
-            if (l != null && !listeners.contains(l)) {
-                listeners.add(l);
-            }
+        if (l != null) {
+            listeners.addIfAbsent(l);
         }
     }
 
     public void removeTreeModelListener(TreeModelListener l) {
-        synchronized (listeners) {
-            if (l != null && listeners.contains(l)) {
-                listeners.remove(l);
-            }
-        }
+        listeners.remove(l);
     }