Changeset 31792 in osm for applications

Timestamp:

2015-12-03T01:02:54+01:00 (9 years ago)

Author:

donvip

Message:

sonar - Performance - Method passes constant String of length 1 to character overridden method

Location:

applications/viewer/jmapviewer/src/org/openstreetmap/gui/jmapviewer

Files:

• Coordinate.java (modified) (1 diff)
• MapMarkerCircle.java (modified) (1 diff)
• MapPolygonImpl.java (modified) (1 diff)
• checkBoxTree/CheckBoxNodeData.java (modified) (3 diffs)

applications/viewer/jmapviewer/src/org/openstreetmap/gui/jmapviewer/Coordinate.java

@@ -57 +57 @@
     @Override
     public String toString() {
-        return "Coordinate[" + data.y + ", " + data.x + "]";
+        return "Coordinate[" + data.y + ", " + data.x + ']';
     }
 

applications/viewer/jmapviewer/src/org/openstreetmap/gui/jmapviewer/MapMarkerCircle.java

@@ -152 +152 @@
     @Override
     public String toString() {
-        return "MapMarker at " + getLat() + " " + getLon();
+        return "MapMarker at " + getLat() + ' ' + getLon();
     }
 

applications/viewer/jmapviewer/src/org/openstreetmap/gui/jmapviewer/MapPolygonImpl.java

@@ -108 +108 @@
     @Override
     public String toString() {
-        return "MapPolygon [points=" + points + "]";
+        return "MapPolygon [points=" + points + ']';
     }
 }

applications/viewer/jmapviewer/src/org/openstreetmap/gui/jmapviewer/checkBoxTree/CheckBoxNodeData.java

@@ -27 +27 @@
 
     public Boolean isSelected() {
-            return layer.isVisible();
+        return layer.isVisible();
     }
 
@@ -35 +35 @@
 
     public String getText() {
-            return layer.getName();
+        return layer.getName();
     }
 
@@ -43 +43 @@
 
     public void setAbstractLayer(final AbstractLayer layer) {
-            this.layer = layer;
+        this.layer = layer;
     }
 
     @Override
     public String toString() {
-            return getClass().getSimpleName() + "[" + getText() + "/" + isSelected() + "]";
+        return getClass().getSimpleName() + '[' + getText() + '/' + isSelected() + ']';
     }
 }