Changeset 6890 in josm for trunk/src/org/openstreetmap/josm/data/osm/history

Timestamp:

2014-02-27T02:50:12+01:00 (10 years ago)

Author:

Don-vip

Message:

fix some Sonar issues (Constructor Calls Overridable Method)

Location:

trunk/src/org/openstreetmap/josm/data/osm/history

Files:

• HistoryNode.java (modified) (2 diffs)
• HistoryOsmPrimitive.java (modified) (1 diff)

trunk/src/org/openstreetmap/josm/data/osm/history/HistoryNode.java

@@ -73 +73 @@
      * @return the coordinates. May be null.
      */
-    public LatLon getCoords() {
+    public final LatLon getCoords() {
         return coords;
     }
@@ -81 +81 @@
      * @param coords the coordinates. Can be null.
      */
-    public void setCoords(LatLon coords) {
+    public final void setCoords(LatLon coords) {
         this.coords = coords;
     }

trunk/src/org/openstreetmap/josm/data/osm/history/HistoryOsmPrimitive.java

@@ -38 +38 @@
     private Map<String, String> tags;
 
-    protected void ensurePositiveLong(long value, String name) {
+    protected final void ensurePositiveLong(long value, String name) {
         if (value <= 0) {
             throw new IllegalArgumentException(MessageFormat.format("Parameter ''{0}'' > 0 expected. Got ''{1}''.", name, value));