Changeset 12476 in osm for applications

Timestamp:

2008-12-22T14:48:12+01:00 (16 years ago)

Author:

hampelratte

Message:

avoid ConcurrentModificationException by copying data.nodes to an array before rendering

File:

• applications/editors/josm/plugins/openstreetbugs/src/org/openstreetmap/josm/plugins/osb/OsbLayer.java (modified) (2 diffs)

applications/editors/josm/plugins/openstreetbugs/src/org/openstreetmap/josm/plugins/osb/OsbLayer.java

@@ -111 +111 @@
         @Override
         public void paint(Graphics g, MapView mv) {
-                for (Node node : data.nodes) {
+            Object[] nodes = data.nodes.toArray();
+                for (int i = 0; i < nodes.length; i++) {
+                    Node node = (Node) nodes[i];
+                    
                         // don't paint deleted nodes
                         if(node.deleted) {
@@ -148 +151 @@
                                         Rectangle2D fontBounds = null;
                                         String[] lines = desc.split("<hr>");
-                                        for (int i = 0; i < lines.length; i++) {
-                                                String line = lines[i];
+                                        for (int j = 0; j < lines.length; j++) {
+                                                String line = lines[j];
                                                 fontBounds = g.getFontMetrics().getStringBounds(line, g);
                                                 tooltipWidth = Math.max(tooltipWidth, (int)fontBounds.getWidth());