Changeset 5416 in josm

Timestamp:

Aug 10, 2012 1:29:09 AM (10 months ago)

Author:

Don-vip

Message:

see #7938 - code optimization (patch by verdy_p)

File:

• trunk/src/org/openstreetmap/josm/gui/dialogs/ConflictDialog.java (modified) (1 diff)

trunk/src/org/openstreetmap/josm/gui/dialogs/ConflictDialog.java

@@ -220 +220 @@
             }
             public void visit(Relation e) {
-                for (RelationMember em : e.getMembers()) {
-                    OsmPrimitive m = em.getMember();
-                    if (m instanceof Node || m instanceof Way) {
-                        m.visit(this);
-                    } else if (m instanceof Relation && !visited.contains(m)) {
-                        visited.add((Relation) m);
-                        try {
-                            m.visit(this);
-                        } finally {
-                            visited.remove(m);
+                if (!visited.contains(e)) {
+                    visited.add(e);
+                    try {
+                        for (RelationMember em : e.getMembers()) {
+                            em.getMember().visit(this);
                         }
+                    } finally {
+                        visited.remove(e);
                     }
                 }