search

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

Addressing some less then great logspamming due to what gets logged #194

Closed ProgrammerDan closed 8 years ago

ProgrammerDan commented 8 years ago

for each config item. Needs testing but should resolve that issue. Also includes minor fix for extremely low probability chance that when breaking a 0 health reinforced block, the reinforcement would drop, due to how Random.nextDouble() works.

Maxopoly commented 8 years ago

Looks good to merge

ProgrammerDan commented 8 years ago

Cool, any chance you can toss it on testing and make sure it doesn't explode in an unexpected way?

ProgrammerDan commented 8 years ago

Also no pressure if not, I'll just do it later tonight if you're busy!

Maxopoly commented 8 years ago

Uh, probably not today

ProgrammerDan commented 8 years ago

Ok, no worries. I'll handle that tonight and merge then. Thanks for the review!