Changeset 19014 in josm for trunk/src/org/openstreetmap/josm/gui/dialogs/relation/actions/ApplyAction.java

Timestamp:

2024-03-08T10:58:03+01:00 (2 months ago)

Author:

GerdP

Message:

fix #23527: Memory leak in relation editor

• add new method wouldRelationBeUseful() in IRelationEditorActionAccess which doesn't call the problematic method getChangedRelation() to evaluate whether a relation would be useful if saved.
• add comments and code in unit test for the method that shows how to handle the result of getChangedRelation() in case it is not needed any longer to avoid the memory leak.

File:

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

trunk/src/org/openstreetmap/josm/gui/dialogs/relation/actions/ApplyAction.java

@@ -36 +36 @@
     @Override
     public void updateEnabledState() {
-        setEnabled(this.editorAccess.getChangedRelation().isUseful() && isEditorDirty());
+        setEnabled(this.editorAccess.wouldRelationBeUseful() && isEditorDirty());
     }
 }