LowMemorySet lag

[MedicOP]

LowMemorySet sorting causes huge lag if the set is big.

The LowMemorySet should be changed back to HashSet IMO.

[MedicOP]

Any updates on this?

Even if you have a million entries, assuming the overhead is as high as 64 bytes per entry (way more than it actually is), that's only 64MB of RAM.

I don't see why this change is necessary, and sorting the set each time an entry is added is horrible for performance.

[electronicboy]

This class has been primed for removal, just; Dreadful mistake of installing Catalina meant that I couldn't make the relevant changes, I'll try and get to this later goes back to huggling mojave

[Brokkonaut]

Doesn't the remaining part of the patch (the String.intern()) create a memory leak, because all names that have ever been added to the Team will never be removed from the memory again?

[Janmm14]

I'm sure the intern list/map is using weak references.