Changes between Initial Version and Version 1 of Ticket #21825, comment 35


Ignore:
Timestamp:
2022-03-17T17:41:02+01:00 (3 years ago)
Author:
taylor.smock

Legend:

Unmodified
Added
Removed
Modified

  • Ticket #21825, comment 35

    initial v1  
    44
    55The actual problem appears to be from the code updating connectivity. I'll have to rework the listeners updating the buttons to avoid being called in the loop (`MemberTableModel#wayNodesChanged`). Most of it is from creating thousands of new relations. :(
     6
     7Edit: See [source:/trunk/src/org/openstreetmap/josm/gui/dialogs/relation/MemberTableModel.java#L178 MemberTableModel.java#L178] -- I'm looking into replacing the `for` loop with a single table update event. This decreases the UI freeze on my machine from ~20s to <1s. And the connection column appears to properly update. I'm checking to see if there are any issues with the change, but I'm not seeing any -- I'll be doing a more in-depth dive to all the listeners that are called by it in the next hour or two.