dkfans
Small Refactor for thing_stats.c
Reordered stuff in thing_stats.c.
Removed boolean argument on compute_creature_max_armour the buff is now checked on calculate_correct_creature_armour.
Added calculate_correct_creature_defense and calculate_correct_creature_dexterity and change few functions accordingly (see creature_states_combt.c - thing_creature.c - thing_shots.c).
Removed dungeon modifier on project_creature_attack_melee_damage and compute_creature_attack_melee_damage as it's no longer required since calculate_correct_creature_strength is now used where it should be (see thing_creature.c).
Note: calculate_correct_creature_defense and calculate_correct_creature_dexterity added because I plan to add a dungeon modifier for them.
Regarding last commit, for modifiers I use TbBool is_neutral_thing to check if owner is neutral, so I added on this function an extra check for case when owner is player 9 so it doesn't try to get its dungeon.
I could add another function if you think it's more appropriate but then I would need to update all the conditions and one could consider player 9 (all players) to be same as neutral. Let me know!
Disregard my last comment, I had to make a new check anyway to avoid potentials problems with unrelated functions using is_neutral_thing so I made a is_valid_player_id for dungeons modifiers.
