Prevent Loaded Pearls Being Free'd

[Ty4Code]

Fixes issue #15. Prevents previously unloaded pearls
from being free'd/expired even if the chunk has been reloaded.

[ttk2]

what exactly was the issue?

On Wed, Nov 19, 2014 at 4:04 PM, MrTwiggy notifications@github.com wrote:

Fixes issue #15
#15. Prevents previously
unloaded pearls

from being free'd/expired even if the chunk has been reloaded.

You can merge this Pull Request by running

git pull https://github.com/MrTwiggy/PrisonPearl master

Or view, comment on, or merge it at:

#17
Commit Summary

• Prevent Loaded Pearls Being Free'd

File Changes

• M src/com/untamedears/PrisonPearl/PrisonPearlManager.java
  https://github.com/Civcraft/PrisonPearl/pull/17/files#diff-0 (14)

Patch Links:

• https://github.com/Civcraft/PrisonPearl/pull/17.patch
• https://github.com/Civcraft/PrisonPearl/pull/17.diff

—
Reply to this email directly or view it on GitHub
#17.

[Ty4Code]

The issue was that if a prison pearl was dropped on the ground, and then the chunk that the pearl was inside was unloaded. The pearl was added to a list, and after a certain delay, the pearl was "free'd" and destroyed, regardless of whether the chunk was re-loaded again.

[ttk2]

ah, so this just cancels the event on reload. Strange I remember a pull to
fix that earlier.

On Wed, Nov 19, 2014 at 4:11 PM, MrTwiggy notifications@github.com wrote:

The issue was that if a prison pearl was dropped on the ground, and then
the chunk that the pearl was inside was unloaded. The pearl was added to a
list, and after a certain delay, the pearl was "free'd" and destroyed,
regardless of whether the chunk was re-loaded again.

—
Reply to this email directly or view it on GitHub
#17 (comment).

[rourke750]

@ttk2 Its a little different. Anyways code is good so I'm merging. Check for a new jar on jenkins and this can go live without testing.