Update IslandLevelCalculator.java #314

tastybento · 2024-05-31T18:06:53Z

Fix for occasional errors likely due to the use of the remove() method within a lambda expression inside the thenAccept method. This lambda expression is executed asynchronously, which means that the iterator may not be in a consistent state when remove() is called.

Since thenAccept runs asynchronously, the iteration over stackedBlocks may continue before the lambda is executed, leading to an inconsistent state when it.remove() is called.

To fix this issue, you should collect the elements to be removed in a separate collection and remove them after the iteration is complete.

In this version, the elements to be removed are collected in a separate List<Location> toRemove, which is synchronized to handle concurrent access. After the iteration is complete and all asynchronous tasks have been executed, the collected elements are removed from stackedBlocks.

Fix for occasional errors likely due to the use of the remove() method within a lambda expression inside the thenAccept method. This lambda expression is executed asynchronously, which means that the iterator may not be in a consistent state when remove() is called.