Fixes some unnecessary capitalization in monsters.json.
don't use reinterpret_cast, use dynamic_cast on objects with virtual inheritance
clean out duplicate & commented out code
astyle
Set scroll position to 0, if all items are in view

Prevent a negative scroll position, because the display loops start at the scroll offset that means they would start with a negative value and accessing that entry in a list causes a crash.
Prevent negative selection indices in worldfactory.cpp

initialize active_header & last_active_header to 0.
move the has container for liquid check to player class
centralize the creation of crafting

Include result_mult into the check for free container space.

Create recipe::create_result to create the resulting item used in checking for enough container space (in case it's a liquid).
Make player::has_container_for check for free container space.
use recipe::create_result in two more places
add display of missing tools/components for recrafting
remove a pointless call to item_controller

and some pointles brackets
Remove player::has_watertight_container and player::has_matching_liquid

they are now superseded by player::has_container_for
Extract valid_mod_locations from json to the translation template
Made some strings from mod manager translatable
erase some warnings and use size_t for iteration
Adds player::add_martialart that prevents addition of already known styles

Use that function in the debug function
fix some rather curious formatting
Fix latest Mac compilation bug

savegame_minver_overmap was apparently completely unused, and the latest version of Xcode throws a fit over unused variables.
Remove commented out variables

These don't look like they'll be used again, so might as well remove them. Having clean code is nice.