@thyraxx opened this Pull Request on September 21st 2021 Contributor

Description

Related issue and possible duplicates?: #467 #442 #432 #233
After rigorously testing it seems that sometimes inside a stream map the variable is null, added null check.

Kept placing blocks mid air and destroying them in creative mode tons of times in quick succession, eventually game crashed with the exact same log from the related issue.

@thyraxx commented on September 21st 2021 Contributor

After looking around further it looks like the array is cleared. It looks like an array.clear is called while the stream is busy, which results in an empty array, hence it becomes null.

We could keep it like this with null check in the stream and it should work fine, I could also change it so the array won't be cleared if it's still busy and remove the null check, or add it on top of the null check. Not sure what the right course of action would be here.

@sudofox commented on September 30th 2021

Thanks for looking into this -- hopefully it gets noticed, since being able to use MobSpawnESP without worrying about it crashing would make my life 10x easier.

This Pull Request was closed on October 8th 2021