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

diff --git a/src/resources/iteminfo.h b/src/resources/iteminfo.h
index 7344d668..5922b6c1 100644
--- a/src/resources/iteminfo.h
+++ b/src/resources/iteminfo.h
@@ -229,8 +229,7 @@ class TaItemInfo: public ItemInfo
     friend class TaItemDB;
 
     public:
-        TaItemInfo()
-        {}
+        TaItemInfo() = default;
 
         // Declare TmwAthena Specific item info here
 };
@@ -246,8 +245,7 @@ namespace ManaServ {
 class ManaServItemInfo: public ItemInfo
 {
     public:
-        ManaServItemInfo()
-        {}
+        ManaServItemInfo() = default;
 
         // Declare Manaserv Specific item info here
 };