Changeset 11878 in josm for trunk/src/org/openstreetmap/josm/data/notes/Note.java

Timestamp:

2017-04-10T00:14:42+02:00 (7 years ago)

Author:

Don-vip

Message:

findbugs - EI_EXPOSE_REP2 + javadoc

File:

• trunk/src/org/openstreetmap/josm/data/notes/Note.java (modified) (6 diffs)

trunk/src/org/openstreetmap/josm/data/notes/Note.java

@@ -11 +11 @@
 
 import org.openstreetmap.josm.data.coor.LatLon;
+import org.openstreetmap.josm.tools.date.DateUtils;
 
 /**
@@ -98 +99 @@
     /** @return Date that this note was submitted */
     public Date getCreatedAt() {
-        return cloneDate(createdAt);
+        return DateUtils.cloneDate(createdAt);
     }
 
@@ -106 +107 @@
      */
     public void setCreatedAt(Date createdAt) {
-        this.createdAt = cloneDate(createdAt);
+        this.createdAt = DateUtils.cloneDate(createdAt);
     }
 
     /** @return Date that this note was closed. Null if it is still open. */
     public Date getClosedAt() {
-        return cloneDate(closedAt);
+        return DateUtils.cloneDate(closedAt);
     }
 
@@ -119 +120 @@
      */
     public void setClosedAt(Date closedAt) {
-        this.closedAt = cloneDate(closedAt);
+        this.closedAt = DateUtils.cloneDate(closedAt);
     }
 
@@ -172 +173 @@
     public void updateWith(Note note) {
         this.comments = note.comments;
-        this.createdAt = cloneDate(note.createdAt);
+        this.createdAt = DateUtils.cloneDate(note.createdAt);
         this.id = note.id;
         this.state = note.state;
@@ -197 +198 @@
         return tr("Note") + ' ' + id + ": " + getFirstComment();
     }
-
-    /**
-     * Null-safe date cloning method.
-     * @param d date to clone, or null
-     * @return cloned date, or null
-     */
-    static Date cloneDate(Date d) {
-        return d != null ? (Date) d.clone() : null;
-    }
 }