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, 5 deletions

diff --git a/src/actor.cpp b/src/actor.cpp
index e1fffaf9..a644c1e0 100644
--- a/src/actor.cpp
+++ b/src/actor.cpp
@@ -22,11 +22,7 @@
 
 #include "map.h"
 
-#include "resources/image.h"
-
-Actor::Actor():
-        mMap(nullptr)
-{}
+Actor::Actor() = default;
 
 Actor::~Actor()
 {