Changeset 2884 in josm for trunk/src/org/openstreetmap

Timestamp:

2010-01-23T16:47:01+01:00 (14 years ago)

Author:

jttt

Message:

Get mouse position in marker layer only when necessary (speed optimization)

File:

• trunk/src/org/openstreetmap/josm/gui/layer/markerlayer/MarkerLayer.java (modified) (1 diff)

trunk/src/org/openstreetmap/josm/gui/layer/markerlayer/MarkerLayer.java

@@ -154 +154 @@
 
     @Override public void paint(Graphics2D g, MapView mv, Bounds box) {
-        boolean mousePressedTmp = mousePressed;
-        Point mousePos = mv.getMousePosition();
         String mkrTextShow = Main.pref.get("marker.show "+getName(), "show");
-
         g.setColor(getColor(getName()));
 
-        for (Marker mkr : data) {
-            if (mousePos != null && mkr.containsPoint(mousePos)) {
-                mkr.paint(g, mv, mousePressedTmp, mkrTextShow);
-                mousePressedTmp = false;
-            } else {
+        if (mousePressed) {
+            boolean mousePressedTmp = mousePressed;
+            Point mousePos = mv.getMousePosition(); // Get mouse position only when necessary (it's the slowest part of marker layer painting)
+            for (Marker mkr : data) {
+                if (mousePos != null && mkr.containsPoint(mousePos)) {
+                    mkr.paint(g, mv, mousePressedTmp, mkrTextShow);
+                    mousePressedTmp = false;
+                }
+            }
+        } else {
+            for (Marker mkr : data) {
                 mkr.paint(g, mv, false, mkrTextShow);
             }