Changeset 1250 in josm for trunk/src/org

Timestamp:

2009-01-11T18:43:37+01:00 (15 years ago)

Author:

ulfl

Message:

isPolygonVisible() shouldn't return true, if width and height are both 0

This indicates we have a way with no nodes in it. Hmmm, happens often in reality, but under which circumstances does this happen? (bug?)

File:

• trunk/src/org/openstreetmap/josm/data/osm/visitor/SimplePaintVisitor.java (modified) (1 diff)

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

@@ -360 +360 @@
     protected boolean isPolygonVisible(Polygon polygon) {
         Rectangle bounds = polygon.getBounds();
+        if (bounds.width == 0 && bounds.height == 0) return false;
         if (bounds.x > nc.getWidth()) return false;
-        else if (bounds.y > nc.getHeight()) return false;
-        else if (bounds.x + bounds.width < 0) return false;
-        else if (bounds.y + bounds.height < 0) return false;
-
+        if (bounds.y > nc.getHeight()) return false;
+        if (bounds.x + bounds.width < 0) return false;
+        if (bounds.y + bounds.height < 0) return false;
         return true;
     }