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/net/manaserv/effecthandler.h b/src/net/manaserv/effecthandler.h
index 268e2725..4ba711d7 100644
--- a/src/net/manaserv/effecthandler.h
+++ b/src/net/manaserv/effecthandler.h
@@ -26,7 +26,7 @@
 
 namespace ManaServ {
 
-class EffectHandler : public MessageHandler
+class EffectHandler final : public MessageHandler
 {
     public:
         EffectHandler();