Changeset 1928 in josm for trunk/src/org/openstreetmap/josm/io

Timestamp:

2009-08-08T11:32:27+02:00 (15 years ago)

Author:

jttt

Message:

Make parameterless RelationMember constructor deprecated

File:

• trunk/src/org/openstreetmap/josm/io/OsmReader.java (modified) (5 diffs)

trunk/src/org/openstreetmap/josm/io/OsmReader.java

@@ -122 +122 @@
         public String type;
         public long id;
-        public RelationMember relationMember;
+        public String role;
     }
 
@@ -204 +204 @@
                         throw new SAXException(tr("Found <member> element in non-relation."));
                     RelationMemberData emd = new RelationMemberData();
-                    emd.relationMember = new RelationMember();
                     String value = atts.getValue("ref");
                     if (value == null)
@@ -220 +219 @@
                     emd.type= value;
                     value = atts.getValue("role");
-                    emd.relationMember.role = value;
+                    emd.role = value;
 
                     if (emd.id == 0)
@@ -393 +392 @@
      * and populate them later.
      */
-    private void createRelations() {
+    private void createRelations() throws SAXException {
 
         // pass 1 - create all relations
@@ -414 +413 @@
 
             for (RelationMemberData emd : e.getValue()) {
-                RelationMember em = emd.relationMember;
+                OsmPrimitive member;
                 if (emd.type.equals("node")) {
-                    em.member = findNode(emd.id);
-                    if (em.member == null) {
-                        em.member = new Node(emd.id);
-                        ds.addPrimitive(em.member);
+                    member = findNode(emd.id);
+                    if (member == null) {
+                        member = new Node(emd.id);
+                        ds.addPrimitive(member);
                     }
                 } else if (emd.type.equals("way")) {
-                    em.member = hm.get(emd.id);
-                    if (em.member == null) {
-                        em.member = findWay(emd.id);
-                    }
-                    if (em.member == null) {
-                        em.member = new Way(emd.id);
-                        ds.addPrimitive(em.member);
+                    member = hm.get(emd.id);
+                    if (member == null) {
+                        member = findWay(emd.id);
+                    }
+                    if (member == null) {
+                        member = new Way(emd.id);
+                        ds.addPrimitive(member);
                     }
                 } else if (emd.type.equals("relation")) {
-                    em.member = findRelation(emd.id);
-                    if (em.member == null) {
-                        em.member = new Relation(emd.id);
-                        ds.addPrimitive(em.member);
+                    member = findRelation(emd.id);
+                    if (member == null) {
+                        member = new Relation(emd.id);
+                        ds.addPrimitive(member);
                     }
                 } else {
-                    // this is an error.
+                    throw new SAXException(tr("Unknown relation member type {0}", emd.type));
                 }
-                en.members.add(em);
+                en.members.add(new RelationMember(emd.role, member));
             }
         }