Fnordsoft/Citadel — pull request #1 by Nebual (2012-04-10 18:11:38+00:00)
Title: Added fire removal checks to BlockBurnEvent

Added fire removal checks to BlockBurnEvent, which slows the effectiveness of a single use of flint and steel versus a citadel building.

In trials, I had a wooden NPC building in which 3 walls were vanilla and one wall was stone citadel'd (10 strength by default config). I set 3 fires on each wall, and after the fires had naturally gone out (~2 minutes), the 3 walls were nearly entirely gone, while the citadel'd wall (with 'extended fire protection') had lost 2-3 durability.

In an identical trial using the existing blockBurn without any particular fire protection, the citadel'd wall continued to burn for 15+ minutes (its still burning) and 80% of blocks were destroyed (notably the top blocks first)

If we added 8 more checks (the stone blocks in http://www.nebtown.info/ss/neb/2012_04_10_10-39-24.png ) we would have aggressive fire resistance, likely a single flintandsteel use against a building would only cause 2-4 blocks to lose 1 durability.

As for benchmarking, 'extended fire protection' approximately doubled the execution time of blockBurn for citadel'd blocks, however as it causes fires to go out faster it may help server performance hehe.