Changeset 2659 in josm for trunk/src/org/openstreetmap/josm/data

Timestamp:

2009-12-19T18:12:03+01:00 (14 years ago)

Author:

jttt

Message:

Minor fixes in mappaint code

Location:

trunk/src/org/openstreetmap/josm/data/osm/visitor

Files:

• MapPaintVisitor.java (modified) (8 diffs)
• SimplePaintVisitor.java (modified) (7 diffs)

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

@@ -11 +11 @@
 import java.awt.Font;
 import java.awt.FontMetrics;
-import java.awt.Graphics2D;
 import java.awt.Image;
 import java.awt.Point;
@@ -92 +91 @@
     public ElemStyle getPrimitiveStyle(OsmPrimitive osm) {
         if(!useStyleCache)
-            return (styles != null) ? styles.get(osm) : null;
-
-            if(osm.mappaintStyle == null && styles != null) {
-                osm.mappaintStyle = styles.get(osm);
-                if(osm instanceof Way) {
-                    ((Way)osm).isMappaintArea = styles.isArea(osm);
-                }
-            }
-            return osm.mappaintStyle;
+            return ((styles != null) ? styles.get(osm) : null);
+
+        if(osm.mappaintStyle == null && styles != null) {
+            osm.mappaintStyle = styles.get(osm);
+            if(osm instanceof Way) {
+                ((Way)osm).isMappaintArea = styles.isArea(osm);
+            }
+        }
+        return osm.mappaintStyle;
     }
 
@@ -255 +254 @@
             width = l.width;
             realWidth = l.realWidth;
-            dashed = l.dashed;
+            dashed = l.getDashed();
             dashedColor = l.dashedColor;
         }
@@ -292 +291 @@
 
         /* draw overlays under the way */
-        if(l != null && l.overlays != null)
-        {
-            for(LineElemStyle s : l.overlays)
-            {
-                if(!s.over)
-                {
+        if(l != null && l.overlays != null) {
+            for(LineElemStyle s : l.overlays) {
+                if(!s.over) {
                     lastN = null;
-                    for(Node n : w.getNodes())
-                    {
-                        if(lastN != null)
-                        {
+                    for(Node n : w.getNodes()) {
+                        if(lastN != null) {
                             drawSeg(lastN, n, s.color != null  && !data.isSelected(w) ? s.color : color,
-                                    false, s.getWidth(width), s.dashed, s.dashedColor);
+                                    false, s.getWidth(width), s.getDashed(), s.dashedColor);
                         }
                         lastN = n;
@@ -338 +332 @@
                         {
                             drawSeg(lastN, n, s.color != null && !data.isSelected(w) ? s.color : color,
-                                    false, s.getWidth(width), s.dashed, s.dashedColor);
+                                    false, s.getWidth(width), s.getDashed(), s.dashedColor);
                         }
                         lastN = n;
@@ -1156 +1150 @@
         int w = icon.getIconWidth(), h=icon.getIconHeight();
         icon.paintIcon ( Main.map.mapView, g, p.x-w/2, p.y-h/2 );
-        if(showNames > dist)
-        {
+        if(showNames > dist && annotate) {
             String name = getNodeName(n);
-            if (name!=null && annotate)
-            {
+            if (name!=null) {
                 if (inactive || n.isDisabled()) {
                     g.setColor(inactiveColor);
@@ -1219 +1211 @@
     protected void displaySegments(Color newColor, int newWidth, float newDash[], Color newDashedColor) {
         if (currentPath != null) {
-            Graphics2D g2d = (Graphics2D)g;
-            g2d.setColor(inactive ? inactiveColor : currentColor);
+            g.setColor(inactive ? inactiveColor : currentColor);
             if (currentStroke == null && useStrokes > dist) {
                 if (currentDashed.length > 0) {
-                    try {
-                        g2d.setStroke(new BasicStroke(currentWidth,BasicStroke.CAP_BUTT,BasicStroke.JOIN_ROUND,0,currentDashed,0));
-                    } catch (IllegalArgumentException e) {
-                        g2d.setStroke(new BasicStroke(currentWidth,BasicStroke.CAP_ROUND,BasicStroke.JOIN_ROUND));
-                    }
+                    g.setStroke(new BasicStroke(currentWidth,BasicStroke.CAP_BUTT,BasicStroke.JOIN_ROUND,0,currentDashed,0));
                 } else {
-                    g2d.setStroke(new BasicStroke(currentWidth,BasicStroke.CAP_ROUND,BasicStroke.JOIN_ROUND));
-                }
-            }
-            g2d.draw(currentPath);
+                    g.setStroke(new BasicStroke(currentWidth,BasicStroke.CAP_ROUND,BasicStroke.JOIN_ROUND));
+                }
+            }
+            g.draw(currentPath);
 
             if(currentDashedColor != null) {
-                g2d.setColor(currentDashedColor);
+                g.setColor(currentDashedColor);
                 if (currentStroke == null && useStrokes > dist) {
                     if (currentDashed.length > 0) {
@@ -1242 +1229 @@
                         currentDashedOffset[currentDashed.length-1] = currentDashed[0];
                         float offset = currentDashedOffset[0];
-                        try {
-                            g2d.setStroke(new BasicStroke(currentWidth,BasicStroke.CAP_BUTT,BasicStroke.JOIN_ROUND,0,currentDashedOffset,offset));
-                        } catch (IllegalArgumentException e) {
-                            g2d.setStroke(new BasicStroke(currentWidth,BasicStroke.CAP_ROUND,BasicStroke.JOIN_ROUND));
-                        }
+                        g.setStroke(new BasicStroke(currentWidth,BasicStroke.CAP_BUTT,BasicStroke.JOIN_ROUND,0,currentDashedOffset,offset));
                     } else {
-                        g2d.setStroke(new BasicStroke(currentWidth,BasicStroke.CAP_ROUND,BasicStroke.JOIN_ROUND));
-                    }
-                }
-                g2d.draw(currentPath);
+                        g.setStroke(new BasicStroke(currentWidth,BasicStroke.CAP_ROUND,BasicStroke.JOIN_ROUND));
+                    }
+                }
+                g.draw(currentPath);
             }
 
             if(useStrokes > dist) {
-                g2d.setStroke(new BasicStroke(1));
+                g.setStroke(new BasicStroke(1));
             }
 

trunk/src/org/openstreetmap/josm/data/osm/visitor/SimplePaintVisitor.java

@@ -8 +8 @@
 import java.awt.BasicStroke;
 import java.awt.Color;
-import java.awt.Graphics;
 import java.awt.Graphics2D;
 import java.awt.Point;
@@ -45 +44 @@
      * The environment to paint to.
      */
-    protected Graphics g;
+    protected Graphics2D g;
     /**
      * MapView to get screen coordinates.
@@ -90 +89 @@
     protected GeneralPath currentPath = new GeneralPath();
 
-    Rectangle bbox = new Rectangle();
-
     public void getColors()
     {
@@ -124 +121 @@
         getColors();
 
-        ((Graphics2D)g).setRenderingHint(RenderingHints.KEY_ANTIALIASING,
+        g.setRenderingHint(RenderingHints.KEY_ANTIALIASING,
                 Main.pref.getBoolean("mappaint.use-antialiasing", false) ?
                         RenderingHints.VALUE_ANTIALIAS_ON : RenderingHints.VALUE_ANTIALIAS_OFF);
@@ -386 +383 @@
                 }
 
-                ((Graphics2D) g).draw(relatedWayStroke.createStrokedShape(path));
+                g.draw(relatedWayStroke.createStrokedShape(path));
             }
         }
@@ -477 +474 @@
     }
 
-    public void setGraphics(Graphics g) {
+    public void setGraphics(Graphics2D g) {
         this.g = g;
     }
@@ -491 +488 @@
         if (currentPath != null) {
             g.setColor(currentColor);
-            ((Graphics2D) g).draw(currentPath);
+            g.draw(currentPath);
             currentPath = new GeneralPath();
             currentColor = newColor;