Changeset 3153 in josm for trunk/src/org/openstreetmap/josm/data/osm/Relation.java

Timestamp:

2010-03-23T21:09:17+01:00 (14 years ago)

Author:

jttt

Message:

Return copy of bbox in Way.getBBox (to make sure internal copy won't be modified)

File:

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

trunk/src/org/openstreetmap/josm/data/osm/Relation.java

@@ -334 +334 @@
     @Override
     public BBox getBBox() {
+        if (members.isEmpty())
+            return new BBox(0, 0, 0, 0);
         if (getDataSet() == null)
             return calculateBBox(new HashSet<PrimitiveId>());
@@ -339 +341 @@
             if (bbox == null) {
                 bbox = calculateBBox(new HashSet<PrimitiveId>());
-                if (bbox == null) {
-                    bbox = new BBox(0, 0, 0, 0); // No members
-                }
-            }
-            return  bbox;
+            }
+            return new BBox(bbox);
         }
     }