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

Timestamp:

2010-11-29T23:13:54+01:00 (14 years ago)

Author:

bastiK

Message:

fixed #5245 - strange lines in wireframe view (activate clipping unconditionally, in MapPaintVisitor this is already the case)

File:

• trunk/src/org/openstreetmap/josm/data/osm/visitor/paint/SimplePaintVisitor.java (modified) (2 diffs)

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

@@ -192 +192 @@
 
         //profilerN = 0;
-        for (final OsmPrimitive osm : data.getSelected())
+        for (final OsmPrimitive osm : data.getSelected()) {
             if (!osm.isDeleted()) {
                 osm.visit(this);
                 //        profilerN++;
             }
+        }
         displaySegments();
 
@@ -469 +470 @@
 
     protected void drawSegment(GeneralPath path, Point p1, Point p2, boolean showDirection) {
-        boolean drawIt = false;
-        if (Main.isOpenjdk) {
-            /**
-             * Work around openjdk bug. It leads to drawing artefacts when zooming in a lot. (#4289, #4424)
-             * (It looks like int overflow when clipping.) We do custom clipping.
-             */
-            Rectangle bounds = g.getClipBounds();
-            bounds.grow(100, 100);                  // avoid arrow heads at the border
-            LineClip clip = new LineClip(p1, p2, bounds);
-            drawIt = clip.execute();
+        Rectangle bounds = g.getClipBounds();
+        bounds.grow(100, 100);                  // avoid arrow heads at the border
+        LineClip clip = new LineClip(p1, p2, bounds);
+        if (clip.execute()) {
             p1 = clip.getP1();
             p2 = clip.getP2();
-        } else {
-            drawIt = isSegmentVisible(p1, p2);
-        }
-        if (drawIt) {
             path.moveTo(p1.x, p1.y);
             path.lineTo(p2.x, p2.y);