Crash in QuadTreeNode annotations(in rect: MKMapRect)

[kylemay-gridstone]

New Issue Checklist

• I updated Cluster to the latest version.
• I read the Contribution Guidelines.
• I read the documentation.
• I searched for existing GitHub issues.

Issue Description

I have noticed a few times that there has been a crash in QuadTree.swift:132
The line is
for annotation in annotations where rect.contains(annotation.coordinate)
and the crash is due to index out of bounds. Presumably this is due to iterating the annotations array while it is being modified. Perhaps a copy of this array should be made inside the method to be used for the iteration. I'm not sure if this fixes it.

I cannot replicate this, however it has happened on multiple versions including the latest, 2.2.9. When this happened today the map was in memory but not visible on screen (page view controller sho8wing another page).

Environment

• iOS Version: 12.0, have seen it on 11 also
• Device(s): iPhone 8, iPad Mini 4

[efremidze]

Thanks for reporting. Currently fixing this issue.

[efremidze]

Released 2.3.0. Let me know if you still see the crash.

[petrovRV]

@efremidze I have the same problem, I updated Cluster to the latest version (Cluster (2.4.0)).
https://drive.google.com/open?id=1JVSrB7vGc71UO7Jm93_5weLcaevItpnb

[efremidze]

I recently noticed it's still occurring. I'll fix it asap.

[antoine20001]

Same issue on Cluster (2.4.0), iPhone XR iOS 12.1

[petrovRV]

Hello. @efremidze. Do you have some news about this problem?

[efremidze]

@petrovRV @antoine20001 Sorry for the delay. I've improved thread safety in 2.4.1. Try it out.