General code cleanups

* Use final for all message handlers, Client, LocalPlayer,
  Being::getType, Being::setPosition and Being::setMap.
  (avoids some warnings about virtual dispatch in constructors)
* Use auto in more places
* Use emplace_back instead of push_back in some places
* Use default member initializers
* Less else after return
* Removed superfluous .c_str()
* Removed type aliases that are only used once
* Removed more unused includes

1 files changed, 1 insertions, 1 deletions

diff --git a/src/map.h b/src/map.h
index 4e054bdb..d03570d8 100644
--- a/src/map.h
+++ b/src/map.h
@@ -69,7 +69,7 @@ class TileAnimation
         ~TileAnimation();
         void update(int ticks = 1);
         void addAffectedTile(MapLayer *layer, int index)
-        { mAffected.push_back(std::make_pair(layer, index)); }
+        { mAffected.emplace_back(layer, index); }
     private:
         std::list<std::pair<MapLayer*, int> > mAffected;
         SimpleAnimation *mAnimation;