Changeset 5798 in josm

Timestamp:

2013-03-23T00:51:05+01:00 (12 years ago)

Author:

Don-vip

Message:

Do not trigger selectionChanged events when selecting the same primitives several times

File:

• trunk/src/org/openstreetmap/josm/data/osm/DataSet.java (modified) (1 diff)

trunk/src/org/openstreetmap/josm/data/osm/DataSet.java

@@ -590 +590 @@
         boolean changed;
         synchronized (selectionLock) {
-            boolean wasEmpty = selectedPrimitives.isEmpty();
+            LinkedHashSet<OsmPrimitive> oldSelection = new LinkedHashSet<OsmPrimitive>(selectedPrimitives);
             selectedPrimitives = new LinkedHashSet<OsmPrimitive>();
-            changed = addSelected(selection, false)
-                    || (!wasEmpty && selectedPrimitives.isEmpty());
+            addSelected(selection, false);
+            changed = !oldSelection.equals(selectedPrimitives);
             if (changed) {
                 selectionSnapshot = null;