Changeset 2402 in josm for trunk


Ignore:
Timestamp:
2009-11-07T18:01:20+01:00 (12 years ago)
Author:
jttt
Message:

Remove some unnecessary calls to fireSelectionChanged()

Location:
trunk/src/org/openstreetmap/josm
Files:
8 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/actions/UploadAction.java

    r2396 r2402  
    669669            //
    670670            layer.cleanupAfterUpload(processedPrimitives);
    671             layer.data.fireSelectionChanged();
    672671            layer.fireDataChange();
    673672            if (lastException != null) {
  • trunk/src/org/openstreetmap/josm/actions/mapmode/DrawAction.java

    r2401 r2402  
    272272    private void tryAgain(MouseEvent e) {
    273273        getCurrentDataSet().setSelected();
    274         Main.main.getCurrentDataSet().fireSelectionChanged();
    275274        mouseClicked(e);
    276275    }
     
    545544            newSelection.add(n);
    546545            ds.setSelected(n);
    547             ds.fireSelectionChanged();
    548546        } else if (!newNode) {
    549547            title = tr("Connect existing way to node");
     
    590588                    (posn0 < selectedWay.getNodesCount()-1) && targetNode.equals(selectedWay.getNode(posn0+1))) {  // next node
    591589                getCurrentDataSet().setSelected(targetNode);
    592                 getCurrentDataSet().fireSelectionChanged();
    593590                lastUsedNode = targetNode;
    594591                return true;
  • trunk/src/org/openstreetmap/josm/actions/mapmode/SelectAction.java

    r2348 r2402  
    520520
    521521        // plain clicks with no modifiers clear the selection
    522         if (!ctrl && !shift)
     522        if (!ctrl && !shift) {
    523523            ds.clearSelection();
     524        }
    524525
    525526        if (ctrl) {
     
    527528            // but Ctrl on an *area* just clears those items
    528529            // out of the selection.
    529             if (area)
     530            if (area) {
    530531                ds.clearSelection(selectionList);
    531             else
     532            } else {
    532533                ds.toggleSelected(selectionList);
     534            }
    533535        } else {
    534536            // This is either a plain click (which means we
     
    538540            ds.addSelected(selectionList);
    539541        }
    540         ds.fireSelectionChanged();
    541542        Main.map.mapView.repaint();
    542543    }
  • trunk/src/org/openstreetmap/josm/data/osm/DataSet.java

    r2401 r2402  
    783783
    784784    public void clenupDeletedPrimitives() {
    785         cleanupDeleted(nodes.iterator());
    786         cleanupDeleted(ways.iterator());
    787         cleanupDeleted(relations.iterator());
    788     }
    789 
    790     private void cleanupDeleted(Iterator<? extends OsmPrimitive> it) {
     785        if (cleanupDeleted(nodes.iterator())
     786                | cleanupDeleted(ways.iterator())
     787                | cleanupDeleted(relations.iterator())) {
     788            fireSelectionChanged();
     789        }
     790    }
     791
     792    private boolean cleanupDeleted(Iterator<? extends OsmPrimitive> it) {
     793        boolean changed = false;
    791794        while (it.hasNext()) {
    792             if (it.next().isDeleted()) {
     795            OsmPrimitive primitive = it.next();
     796            if (primitive.isDeleted()) {
     797                selectedPrimitives.remove(primitive);
     798                allPrimitives.remove(primitive);
     799                changed = true;
    793800                it.remove();
    794801            }
    795802        }
     803        return changed;
    796804    }
    797805
  • trunk/src/org/openstreetmap/josm/gui/MapStatus.java

    r2348 r2402  
    333333                }
    334334            }
    335             ds.fireSelectionChanged();
    336335        }
    337336
  • trunk/src/org/openstreetmap/josm/gui/MapView.java

    r2395 r2402  
    529529            if (getCurrentDataSet() != null) {
    530530                getCurrentDataSet().setSelected();
    531                 getCurrentDataSet().fireSelectionChanged();
    532531            }
    533532        }
  • trunk/src/org/openstreetmap/josm/gui/dialogs/RelationListDialog.java

    r2381 r2402  
    518518            }
    519519            Main.map.mapView.getEditLayer().data.setSelected(selection);
    520             Main.map.mapView.getEditLayer().data.fireSelectionChanged();
    521520        }
    522521
  • trunk/src/org/openstreetmap/josm/gui/io/UploadLayerTask.java

    r2396 r2402  
    143143            return;
    144144        layer.cleanupAfterUpload(processedPrimitives);
    145         layer.data.fireSelectionChanged();;
    146145        layer.fireDataChange();
    147146        layer.onPostUploadToServer();
Note: See TracChangeset for help on using the changeset viewer.