Changeset 478 in josm for trunk/src/org/openstreetmap/josm/gui

Timestamp:

2007-12-01T13:58:58+01:00 (16 years ago)

Author:

gebner

Message:

Speed up merging.

• Don't scan our dataset for every primitive we merge. Use hashtables for that (at least for identical primitives for now).
• Reverse the order we merge the layers when pressing the merge button in the layers list, so that when you merge 25 layers by pressing the merge button 25 times, you merge a small layer into a bigger one, and not the other way round.

File:

• trunk/src/org/openstreetmap/josm/gui/dialogs/LayerListDialog.java (modified) (2 diffs)

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

@@ -251 +251 @@
                 buttonPanel.add(deleteButton);
 
-                mergeButton.setToolTipText(tr("Merge the selected layer into the layer directly below."));
+                mergeButton.setToolTipText(tr("Merge the layer directly below into the selected layer."));
                 mergeButton.addActionListener(new ActionListener(){
                         public void actionPerformed(ActionEvent e) {
-                                Layer lFrom = (Layer)instance.getSelectedValue();
-                                Layer lTo = (Layer)model.get(instance.getSelectedIndex()+1);
+                                Layer lTo = (Layer)instance.getSelectedValue();
+                                Layer lFrom = (Layer)model.get(instance.getSelectedIndex()+1);
                                 lTo.mergeFrom(lFrom);
-                                instance.setSelectedValue(lTo, true);
                                 mapView.removeLayer(lFrom);
+                                updateButtonEnabled();
+                                mapView.repaint();
                         }
                 });
@@ -277 +278 @@
                 boolean enable = model.getSize() > 1;
                 enable = enable && sel < model.getSize()-1;
-                enable = enable && l.isMergable((Layer)model.get(sel+1));
+                enable = enable && ((Layer)model.get(sel+1)).isMergable(l);
                 mergeButton.setEnabled(enable);
                 upButton.setEnabled(sel > 0);