Ignore:
Timestamp:
2016-12-07T23:47:56+01:00 (7 years ago)
Author:
Don-vip
Message:

fix #14090 - use DefaultListSelectionModel adjusting selection mode to avoid too much costly listener calls

Location:
trunk/src/org/openstreetmap/josm/gui/dialogs
Files:
7 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/org/openstreetmap/josm/gui/dialogs/ConflictDialog.java

    r11357 r11365  
    313313    @Override
    314314    public void selectionChanged(Collection<? extends OsmPrimitive> newSelection) {
     315        lstConflicts.setValueIsAdjusting(true);
    315316        lstConflicts.clearSelection();
    316317        for (OsmPrimitive osm : newSelection) {
    … …  
    322323            }
    323324        }
     325        lstConflicts.setValueIsAdjusting(false);
    324326    }
    325327

  • trunk/src/org/openstreetmap/josm/gui/dialogs/LayerListDialog.java

    r10824 r11365  
    841841            }
    842842            fireTableDataChanged();
     843            selectionModel.setValueIsAdjusting(true);
    843844            selectionModel.clearSelection();
    844845            for (int row : sel) {
    845846                selectionModel.addSelectionInterval(row-1, row-1);
    846847            }
     848            selectionModel.setValueIsAdjusting(false);
    847849            ensureSelectedIsVisible();
    848850        }
    … …  
    874876            }
    875877            fireTableDataChanged();
     878            selectionModel.setValueIsAdjusting(true);
    876879            selectionModel.clearSelection();
    877880            for (int row : sel) {
    878881                selectionModel.addSelectionInterval(row+1, row+1);
    879882            }
     883            selectionModel.setValueIsAdjusting(false);
    880884            ensureSelectedIsVisible();
    881885        }

  • trunk/src/org/openstreetmap/josm/gui/dialogs/MapPaintDialog.java

    r10694 r11365  
    351351            int[] pos = tblStyles.getSelectedRows();
    352352            MapPaintStyles.toggleStyleActive(pos);
     353            selectionModel.setValueIsAdjusting(true);
    353354            selectionModel.clearSelection();
    354355            for (int p: pos) {
    355356                selectionModel.addSelectionInterval(p, p);
    356357            }
     358            selectionModel.setValueIsAdjusting(false);
    357359        }
    358360    }
    … …  
    387389            MapPaintStyles.moveStyles(sel, increment);
    388390
     391            selectionModel.setValueIsAdjusting(true);
    389392            selectionModel.clearSelection();
    390393            for (int row: sel) {
    391394                selectionModel.addSelectionInterval(row + increment, row + increment);
    392395            }
     396            selectionModel.setValueIsAdjusting(false);
    393397            model.ensureSelectedIsVisible();
    394398        }
    … …  
    434438            MapPaintStyles.reloadStyles(rows);
    435439            Main.worker.submit(() -> SwingUtilities.invokeLater(() -> {
     440                selectionModel.setValueIsAdjusting(true);
    436441                selectionModel.clearSelection();
    437442                for (int r: rows) {
    438443                    selectionModel.addSelectionInterval(r, r);
    439444                }
     445                selectionModel.setValueIsAdjusting(false);
    440446            }));
    441447        }

  • trunk/src/org/openstreetmap/josm/gui/dialogs/RelationListDialog.java

    r11357 r11365  
    149149        filter = setupFilter();
    150150
    151         displaylist.addListSelectionListener(e -> updateActionsRelationLists());
     151        displaylist.addListSelectionListener(e -> {
     152            if (!e.getValueIsAdjusting()) updateActionsRelationLists();
     153        });
    152154
    153155        // Setup popup menu handler
    … …  
    563565         */
    564566        public void setSelectedRelations(Collection<Relation> sel) {
     567            selectionModel.setValueIsAdjusting(true);
    565568            selectionModel.clearSelection();
    566             if (sel == null || sel.isEmpty())
    567                 return;
    568             if (!getVisibleRelations().containsAll(sel)) {
    569                 resetFilter();
    570             }
    571             for (Relation r: sel) {
    572                 Integer i = getVisibleRelationIndex(r);
    573                 if (i != null) {
    574                     selectionModel.addSelectionInterval(i, i);
    575                 }
    576             }
     569            if (sel != null && !sel.isEmpty()) {
     570                if (!getVisibleRelations().containsAll(sel)) {
     571                    resetFilter();
     572                }
     573                for (Relation r: sel) {
     574                    Integer i = getVisibleRelationIndex(r);
     575                    if (i != null) {
     576                        selectionModel.addSelectionInterval(i, i);
     577                    }
     578                }
     579            }
     580            selectionModel.setValueIsAdjusting(false);
    577581        }
    578582

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

    r11177 r11365  
    578578         */
    579579        public synchronized void setSelected(Collection<OsmPrimitive> sel) {
     580            selectionModel.setValueIsAdjusting(true);
    580581            selectionModel.clearSelection();
    581             if (sel == null) return;
    582             for (OsmPrimitive p: sel) {
    583                 int i = selection.indexOf(p);
    584                 if (i >= 0) {
    585                     selectionModel.addSelectionInterval(i, i);
    586                 }
    587             }
     582            if (sel != null) {
     583                for (OsmPrimitive p: sel) {
     584                    int i = selection.indexOf(p);
     585                    if (i >= 0) {
     586                        selectionModel.addSelectionInterval(i, i);
     587                    }
     588                }
     589            }
     590            selectionModel.setValueIsAdjusting(false);
    588591        }
    589592

  • trunk/src/org/openstreetmap/josm/gui/dialogs/changeset/ChangesetCacheManagerModel.java

    r10647 r11365  
    104104     */
    105105    public void setSelectedChangesets(Collection<Changeset> selected) {
    106         GuiHelper.runInEDTAndWait(selectionModel::clearSelection);
    107         if (selected == null || selected.isEmpty())
    108             return;
    109         for (Changeset cs: selected) {
    110             final int idx = data.indexOf(cs);
    111             if (idx >= 0) {
    112                 GuiHelper.runInEDTAndWait(() -> selectionModel.addSelectionInterval(idx, idx));
     106        selectionModel.setValueIsAdjusting(true);
     107        selectionModel.clearSelection();
     108        if (selected != null) {
     109            for (Changeset cs: selected) {
     110                final int idx = data.indexOf(cs);
     111                if (idx >= 0) {
     112                    selectionModel.addSelectionInterval(idx, idx);
     113                }
    113114            }
    114115        }
     116        GuiHelper.runInEDTAndWait(() -> selectionModel.setValueIsAdjusting(false));
    115117    }
    116118

  • trunk/src/org/openstreetmap/josm/gui/dialogs/changeset/ChangesetListModel.java

    r10647 r11365  
    5050
    5151    public void setSelectedChangesets(Collection<Changeset> changesets) {
     52        selectionModel.setValueIsAdjusting(true);
    5253        selectionModel.clearSelection();
    53         if (changesets == null) return;
    54         for (Changeset cs: changesets) {
    55             int idx = data.indexOf(cs);
    56             if (idx < 0) {
    57                 continue;
    58             }
    59             selectionModel.addSelectionInterval(idx, idx);
    60         }
     54        if (changesets != null) {
     55            for (Changeset cs: changesets) {
     56                int idx = data.indexOf(cs);
     57                if (idx >= 0) {
     58                    selectionModel.addSelectionInterval(idx, idx);
     59                }
     60            }
     61        }
     62        selectionModel.setValueIsAdjusting(false);
    6163    }
    6264
Note: See TracChangeset for help on using the changeset viewer.