Fix for reinforcement maturation.

Civcraft / Citadel

Do not open issues here; open them on the maintained fork @ DevotedMC

https://github.com/DevotedMC/Citadel

BSD 3-Clause "New" or "Revised" License

6 stars 23 forks source link

Fix for reinforcement maturation. #147

Closed ProgrammerDan closed 8 years ago

ProgrammerDan commented 8 years ago

Prior was doing floating math using integers, Java truncates early and often with integer math so was always returning 0 leading to divide-by-zero exceptions. Plus the formula was wrong.

Fixed. Note this is a net-zero change for Civcraft's use (since we don't use maturation) but servers like Devoted or others that are trying to use maturation, this will save them a lot of headache.

ProgrammerDan commented 8 years ago

Should mention this fix has been running live on Devoted for about a week, it's a safe merge.

rourke750 commented 8 years ago

You're good to merge.