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, 2 insertions, 2 deletions

diff --git a/src/net/manaserv/playerhandler.h b/src/net/manaserv/playerhandler.h
index e2a52d4d..a14e3ed5 100644
--- a/src/net/manaserv/playerhandler.h
+++ b/src/net/manaserv/playerhandler.h
@@ -30,13 +30,13 @@
 
 namespace ManaServ {
 
-struct RespawnRequestListener : public gcn::ActionListener
+struct RespawnRequestListener final : public gcn::ActionListener
 {
     void action(const gcn::ActionEvent &event) override;
 };
 static RespawnRequestListener respawnListener;
 
-class PlayerHandler : public MessageHandler, public Net::PlayerHandler
+class PlayerHandler final : public MessageHandler, public Net::PlayerHandler
 {
     public:
         PlayerHandler();