Fixed NPE in yCoordHelper, Other Bugs, Reimplemented Limbo Decay #51

2013-07-24T19:38:59Z

SenseiKiwi commented

2013-07-24 19:38:59 +00:00

(Migrated from github.com)

I believe I've fixed the cause of a NullPointerException that some
players have been experiencing. It seems to have been caused by
unregistered block IDs being present in Limbo. Possibly because people
removed mods and they had generated structures in Limbo. Or the players
had placed blocks and then removed the mods, or changed block IDs
around.

This fixes issue #47.

Also fixed a few other bugs around the code. I found that there were simple ways to circumvent Limbo decay as it was implemented, so I reorganized the code in CommonTickHandler to be more readable and wrote a new implemention of Limbo decay. Most of the logic is in its own class, LimboDecay. Please read the commit for more details.

I believe I've fixed the cause of a NullPointerException that some players have been experiencing. It seems to have been caused by unregistered block IDs being present in Limbo. Possibly because people removed mods and they had generated structures in Limbo. Or the players had placed blocks and then removed the mods, or changed block IDs around. This fixes issue #47. --- Also fixed a few other bugs around the code. I found that there were simple ways to circumvent Limbo decay as it was implemented, so I reorganized the code in CommonTickHandler to be more readable and wrote a new implemention of Limbo decay. Most of the logic is in its own class, LimboDecay. Please read the commit for more details.

Pull request closed

Please reopen this pull request to perform a merge.

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: apex/DimDoors#51