Changeset 8367 in josm for trunk

Timestamp:

2015-05-16T16:43:46+02:00 (9 years ago)

Author:

Don-vip

Message:

fix synchronization issue seen in unit test

File:

• trunk/src/org/openstreetmap/josm/gui/dialogs/SelectionListDialog.java (modified) (7 diffs)

trunk/src/org/openstreetmap/josm/gui/dialogs/SelectionListDialog.java

@@ -493 +493 @@
          * @return a summary of the current JOSM selection
          */
-        public String getJOSMSelectionSummary() {
+        public synchronized String getJOSMSelectionSummary() {
             if (selection.isEmpty()) return tr("Selection");
             int numNodes = 0;
@@ -545 +545 @@
 
         @Override
-        public OsmPrimitive getElementAt(int index) {
+        public synchronized OsmPrimitive getElementAt(int index) {
             return selection.get(index);
         }
 
         @Override
-        public int getSize() {
+        public synchronized int getSize() {
             return selection.size();
         }
@@ -560 +560 @@
          * @return choosen elements in the view
          */
-        public Collection<OsmPrimitive> getSelected() {
+        public synchronized Collection<OsmPrimitive> getSelected() {
             Set<OsmPrimitive> sel = new HashSet<>();
             for(int i=0; i< getSize();i++) {
@@ -575 +575 @@
          * @param sel the collection of primitives to select
          */
-        public void setSelected(Collection<OsmPrimitive> sel) {
+        public synchronized void setSelected(Collection<OsmPrimitive> sel) {
             selectionModel.clearSelection();
             if (sel == null) return;
@@ -598 +598 @@
          * @param selection the collection of currently selected OSM objects
          */
-        public void setJOSMSelection(final Collection<? extends OsmPrimitive> selection) {
+        public synchronized void setJOSMSelection(final Collection<? extends OsmPrimitive> selection) {
             this.selection.clear();
             if (selection != null) {
@@ -636 +636 @@
          * @param toUpdate the collection of primitives to update
          */
-        public void update(Collection<? extends OsmPrimitive> toUpdate) {
+        public synchronized void update(Collection<? extends OsmPrimitive> toUpdate) {
             if (toUpdate == null) return;
             if (toUpdate.isEmpty()) return;
@@ -652 +652 @@
          * Sorts the current elements in the selection
          */
-        public void sort() {
+        public synchronized void sort() {
             if (this.selection.size() <= Main.pref.getInteger("selection.no_sort_above", 100000)) {
                 boolean quick = this.selection.size() > Main.pref.getInteger("selection.fast_sort_above", 10000);