Changeset 2455 in josm for trunk/src/org/openstreetmap/josm/data

Timestamp:

2009-11-15T11:04:06+01:00 (14 years ago)

Author:

jttt

Message:

Do not create new instance of User for every OsmPrimitive, share instances for the same user

File:

• trunk/src/org/openstreetmap/josm/data/osm/User.java (modified) (2 diffs)

trunk/src/org/openstreetmap/josm/data/osm/User.java

@@ -1 +1 @@
 // License: GPL. Copyright 2007 by Immanuel Scholz and others
 package org.openstreetmap.josm.data.osm;
+
+import static org.openstreetmap.josm.tools.I18n.tr;
 
 import java.util.ArrayList;
@@ -48 +50 @@
      */
     public static User  createOsmUser(long uid, String name) {
-        User user = new User(uid, name);
-        userMap.put(user.getId(), user);
+        User user = userMap.get(uid);
+        if (user == null) {
+            user = new User(uid, name);
+            userMap.put(user.getId(), user);
+        }
+        if (!user.getName().equals(name))
+            throw new DataIntegrityProblemException(tr("User with the same uid but different name found"));
         return user;
     }