Changeset 6296 in josm for trunk/src/org/openstreetmap/josm/data/osm/visitor/paint

Timestamp:

2013-10-04T03:27:01+02:00 (11 years ago)

Author:

Don-vip

Message:

Sonar/Findbugs - Avoid commented-out lines of code, javadoc

Location:

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

Files:

• StyledMapRenderer.java (modified) (4 diffs)
• relations/Multipolygon.java (modified) (3 diffs)

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

@@ -498 +498 @@
             } else {
                 TexturePaint texture = new TexturePaint(fillImage.getImage(),
-                        //                        new Rectangle(polygon.xpoints[0], polygon.ypoints[0], fillImage.getWidth(), fillImage.getHeight()));
                         new Rectangle(0, 0, fillImage.getWidth(), fillImage.getHeight()));
                 g.setPaint(texture);
@@ -511 +510 @@
 
         if (text != null && isShowNames()) {
-            /*
-             * abort if we can't compose the label to be rendered
-             */
+            // abort if we can't compose the label to be rendered
             if (text.labelCompositionStrategy == null) return;
             String name = text.labelCompositionStrategy.compose(osm);
@@ -522 +519 @@
             Rectangle2D nb = fontMetrics.getStringBounds(name, g); // if slow, approximate by strlen()*maxcharbounds(font)
 
-            // Point2D c = getCentroid(polygon);
             // Using the Centroid is Nicer for buildings like: +--------+
             // but this needs to be fast.  As most houses are  |   42   |
@@ -528 +524 @@
             // will have to do.                                    ++
             // Centroids are not optimal either, just imagine a U-shaped house.
-            // Point2D c = new Point2D.Double(pb.x + pb.width / 2.0, pb.y + pb.height / 2.0);
-            // Rectangle2D.Double centeredNBounds =
-            //     new Rectangle2D.Double(c.getX() - nb.getWidth()/2,
-            //                            c.getY() - nb.getHeight()/2,
-            //                            nb.getWidth(),
-            //                            nb.getHeight());
 
             Rectangle centeredNBounds = new Rectangle(pb.x + (int)((pb.width - nb.getWidth())/2.0),

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

@@ -488 +488 @@
                         }
                     }
-                } /* for(i = ... */
-            } /* while(joined) */
+                }
+            }
 
             if (nodes == null) {
@@ -497 +497 @@
 
             result.add(new JoinedWay(nodes, wayIds, selected));
-        } /* while(left != 0) */
+        }
 
         return result;
@@ -575 +575 @@
         return innerWays;
     }
-/*
-    public List<PolyData> getInnerPolygons() {
-        return innerPolygons;
-    }
-
-    public List<PolyData> getOuterPolygons() {
-        return outerPolygons;
-    }
-*/
+
     public List<PolyData> getCombinedPolygons() {
         return combinedPolygons;