Ticket #15408: perf-v2.patch

plugins/download_along/build.xml

@@ -3 +3 @@
     <!-- enter the SVN commit message -->
     <property name="commit.message" value="Changed the constructor signature of the plugin main class"/>
     <!-- enter the *lowest* JOSM version this plugin is currently compatible with -->
-    <property name="plugin.main.version" value="10657"/>
+    <property name="plugin.main.version" value="12643"/>
 
         <!-- Configure these properties (replace "..." accordingly).
          See https://josm.openstreetmap.de/wiki/DevelopersGuide/DevelopingPlugins

plugins/download_along/src/org/openstreetmap/josm/plugin/download_along/DownloadAlong.java

@@ -1 +1 @@
 // License: GPL. For details, see LICENSE file.
 package org.openstreetmap.josm.plugin.download_along;
 
-import org.openstreetmap.josm.Main;
+import org.openstreetmap.josm.gui.MainApplication;
 import org.openstreetmap.josm.gui.MainMenu;
 import org.openstreetmap.josm.plugins.Plugin;
 import org.openstreetmap.josm.plugins.PluginInformation;
@@ -10 +10 @@
 
     public DownloadAlong(PluginInformation info) {
         super(info);
-        MainMenu.add(Main.main.menu.moreToolsMenu, new DownloadAlongWayAction());
+        MainMenu.add(MainApplication.getMenu().moreToolsMenu, new DownloadAlongWayAction());
     }
 }

plugins/download_along/src/org/openstreetmap/josm/plugin/download_along/DownloadAlongWayAction.java

@@ -6 +6 @@
 import java.awt.event.ActionEvent;
 import java.awt.event.KeyEvent;
 import java.awt.geom.Area;
+import java.awt.geom.Path2D;
 import java.awt.geom.Rectangle2D;
 import java.util.ArrayList;
 import java.util.Collection;
@@ -22 +23 @@
 import org.openstreetmap.josm.gui.help.HelpUtil;
 import org.openstreetmap.josm.gui.layer.gpx.DownloadAlongPanel;
 import org.openstreetmap.josm.gui.progress.NullProgressMonitor;
+import org.openstreetmap.josm.tools.Logging;
 import org.openstreetmap.josm.tools.Shortcut;
 
 class DownloadAlongWayAction extends DownloadAlongAction {
@@ -83 +85 @@
         double buffer_y = buffer_dist / 100000.0;
         double buffer_x = buffer_y / scale;
         double max_area = panel.getArea() / 10000.0 / scale;
-        Area a = new Area();
+        Path2D path = new Path2D.Double();
         Rectangle2D r = new Rectangle2D.Double();
 
         /*
@@ -100 +102 @@
                 if (previous != null && c.greatCircleDistance(previous) > buffer_dist) {
                     Double d = c.greatCircleDistance(previous) / buffer_dist;
                     int nbNodes = d.intValue();
-                    Main.info(tr("{0} intermediate nodes to download.", nbNodes));
-                    Main.info(tr("between {0} {1} and {2} {3}", c.lat(), c.lon(), previous.lat(),
+                    
+                    Logging.info(tr("{0} intermediate nodes to download.", nbNodes));
+                    Logging.info(tr("between {0} {1} and {2} {3}", c.lat(), c.lon(), previous.lat(),
                             previous.lon()));
                     for (int i = 1; i < nbNodes; i++) {
                         intermediateNodes.add(new LatLon(previous.lat()
                                 + (i * (c.lat() - previous.lat()) / (nbNodes + 1)), previous.lon()
                                 + (i * (c.lon() - previous.lon()) / (nbNodes + 1))));
-                        Main.info(tr("  adding {0} {1}", previous.lat()
+                        Logging.info(tr("  adding {0} {1}", previous.lat()
                                 + (i * (c.lat() - previous.lat()) / (nbNodes + 1)), previous.lon()
                                 + (i * (c.lon() - previous.lon()) / (nbNodes + 1))));
                     }
@@ -117 +120 @@
                     if (previous == null || d.greatCircleDistance(previous) > buffer_dist) {
                         // we add a buffer around the point.
                         r.setRect(d.lon() - buffer_x, d.lat() - buffer_y, 2 * buffer_x, 2 * buffer_y);
-                        a.add(new Area(r));
+//                        a.add(new Area(r));
+                        path.append(r, false);
                         previous = d;
                     }
                 }
@@ -124 +128 @@
                 previous = c;
             }
         }
-        
+        Area a = new Area(path);
         confirmAndDownloadAreas(a, max_area, panel.isDownloadOsmData(), panel.isDownloadGpxData(), 
                 tr("Download from OSM along selected ways"), NullProgressMonitor.INSTANCE);
     }