I think it make sense for MapListContainers to only update the selected map once if a different map is being selected otherwise it's selection listener would be interrupted shortly. This would benefit both memory and performance to avoid constantly allocating new mapInfo and mapConfig pinpointed to the same map; and to avoid a sudden lag spike from spam-clicking a large map.
I think it make sense for MapListContainers to only update the selected map once if a different map is being selected otherwise it's selection listener would be interrupted shortly. This would benefit both memory and performance to avoid constantly allocating new mapInfo and mapConfig pinpointed to the same map; and to avoid a sudden lag spike from spam-clicking a large map.