Ticket #5627: optimize-speed.patch

src/org/openstreetmap/josm/gui/mappaint/SimpleNodeElemStyle.java

@@ -32 +32 @@
         } else {
 
             Color color;
+            boolean isConnection = n.isConnectionNode();
 
             if (painter.isInactive() || n.isDisabled()) {
                 color = settings.getInactiveColor();
             } else if (selected) {
                 color = member ? settings.getRelationSelectedColor() : settings.getSelectedColor();
-            } else if (n.isConnectionNode()) {
+            } else if (isConnection) {
                 if (n.isTagged()) {
                     color = settings.getTaggedConnectionColor();
                 } else {
@@ -53 +54 @@
 
             final int size = max((selected ? settings.getSelectedNodeSize() : 0),
                                     (n.isTagged() ? settings.getTaggedNodeSize() : 0),
-                                    (n.isConnectionNode() ? settings.getConnectionNodeSize() : 0),
+                                    (isConnection ? settings.getConnectionNodeSize() : 0),
                                     settings.getUnselectedNodeSize());
 
             final boolean fill = (selected && settings.isFillSelectedNode()) ||
                                     (n.isTagged() && settings.isFillTaggedNode()) ||
-                                    (n.isConnectionNode() && settings.isFillConnectionNode()) ||
+                                    (isConnection && settings.isFillConnectionNode()) ||
                                     settings.isFillUnselectedNode();
 
             painter.drawNode(n, color, size, fill, name);

src/org/openstreetmap/josm/data/osm/Node.java

@@ -233 +233 @@
     }
 
     public boolean isConnectionNode() {
-        return (OsmPrimitive.getFilteredList(getReferrers(), Way.class)).size() > 1;
+        return isReferredByWays(2);
     }
 
     public String get3892DebugInfo() {

src/org/openstreetmap/josm/data/osm/OsmPrimitive.java

@@ -1285 +1285 @@
         return result;
     }
 
+    /**
+      Return true, if this primitive is referred by at least n ways
+      @param n Minimal number of ways to return true. Must be positive
+     */
+    public final boolean isReferredByWays(int n) {
+        // Count only referrers that are members of the same dataset (primitive can have some fake references, for example
+        // when way is cloned
+        Object referrers = this.referrers;
+        if (referrers == null) return false;
+        checkDataset();
+        if (referrers instanceof OsmPrimitive) {
+          if (!(referrers instanceof Way)) return false;
+          OsmPrimitive ref = (OsmPrimitive)referrers;
+          if (ref.dataSet == dataSet) {
+            return (n<=1);
+          }
+          return false;
+        } else {
+          int counter=0;
+          for (OsmPrimitive o:(OsmPrimitive[])referrers) {
+            if (dataSet == o.dataSet && o instanceof Way) {
+              counter++;
+            }
+          }
+          return counter>=n;
+        }
+    }
+
+
     /*-----------------
      * OTHER METHODS
      *----------------/

src/org/openstreetmap/josm/data/osm/visitor/paint/MapPainter.java

@@ -215 +215 @@
      */
     public void drawNode(Node n, Color color, int size, boolean fill, String name) {
         if (size > 1) {
-            int radius = size / 2;
             Point p = nc.getPoint(n);
-            if ((p.x < 0) || (p.y < 0) || (p.x > nc.getWidth())
-                    || (p.y > nc.getHeight()))
-                return;
+            if ((p.x < 0) || (p.y < 0) || (p.x > nc.getWidth()) || (p.y > nc.getHeight())) return;
+            int radius = size / 2;
 
             if (inactive || n.isDisabled()) {
                 g.setColor(inactiveColor);
@@ -227 +225 @@
                 g.setColor(color);
             }
             if (fill) {
-                g.fillRect(p.x - radius, p.y - radius, size, size);
-                g.drawRect(p.x - radius, p.y - radius, size, size);
+                g.fillRect(p.x - radius, p.y - radius, size + 1, size + 1);
             } else {
                 g.drawRect(p.x - radius, p.y - radius, size, size);
             }

src/org/openstreetmap/josm/data/osm/visitor/paint/MapPaintVisitor.java

@@ -110 +110 @@
 
     public void drawNode(Node n) {
         /* check, if the node is visible at all */
-        if((n.getEastNorth().east()  > maxEN.east() ) ||
-                (n.getEastNorth().north() > maxEN.north()) ||
-                (n.getEastNorth().east()  < minEN.east() ) ||
-                (n.getEastNorth().north() < minEN.north()))
+        EastNorth en=n.getEastNorth();
+        if((en.east()  > maxEN.east() ) ||
+                (en.north() > maxEN.north()) ||
+                (en.east()  < minEN.east() ) ||
+                (en.north() < minEN.north()))
             return;
 
         ElemStyle nodeStyle = getPrimitiveStyle(n, false);