Changeset 17365 in josm for trunk/src/org

Timestamp:

2020-11-27T10:36:45+01:00 (3 years ago)

Author:

GerdP

Message:

see #19885: memory leak with "temporary" objects in validator and actions

• revert fix of counter of relations, it is used in a popup and should in fact count all relations

File:

• trunk/src/org/openstreetmap/josm/command/SplitWayCommand.java (modified) (2 diffs)

trunk/src/org/openstreetmap/josm/command/SplitWayCommand.java

@@ -452 +452 @@
             }
 
+            numberOfRelations++;
             boolean isSimpleCase = true;
 
@@ -562 +563 @@
                 }
             }
-            if (!isSimpleCase)
-                numberOfRelations++;
             if (c != null && isSimpleCase) {
                 if (!r.getMembers().equals(c.getMembers())) {
                     commandList.add(new ChangeMembersCommand(r, new ArrayList<>(c.getMembers())));
-                    numberOfRelations++;
                 }
                 c.setMembers(null); // see #19885