Mostly formatting and wording improvements. Also added a longer
description for the namespace explaining IPFs, image formats and the
image cache briefly.
Apparently, even on Boost 1.73 without Boost placeholders specialized as is_placeholder, they were somehow
getting used in the global namespace. Would explain all the "Boost placeholders in the global namespace is
deprecated" warnings I was getting after 23d1db043f9f8266c77c6270ed4ef3eafbc0cf67.
When testing with BOOST_BIND_NO_PLACEHOLDERS, even though I had `using namespace std::placeholders` in utils/functional.hpp,
compilation still failed in places. This confirms even more that Boost global placeholders were being used. Honestly,
it was simplest just to specify std::placeholders for everything. This also means we can remove the hack in utils/functional.hpp
designed to allow Boost placeholders to work with `std::bind`.
These methods are actually the compiler's defaults. We do need to keep
them around in some capacity because of the explicitly defined default
constructors, so just explicitly default them instead of keeping extra
redundant code around.
This is the same method used by the loading screen. Basically, connection::poll could throw and that exception
wasn't caught in the worker. Instead, it got stored in the std::future object, but was never accessed and therefor
never propagated.
The stacked_widget builder's grid builder stack is only passed to stacked_widget::finalize.
As of 4ada05a2cf74227ec460010db52acf525dddfcf6, the builder_grid is now passed by reference
to the generator's create functions rather than passing another shared_ptr.
Bats deal 3 damage.
Malin heals also 3 damage if he hits (unless the enemy has less than 6 health)
With 28 hp, Malin can take 9 more hits than he does,
compared to 6 previously on hard (and 8 previously on normal)
closes#5296
Most of what used to be in these files (the _t types) were removed once we switched to C++14.
These two value aliases aren't used, so no need to keep them around.
