General code cleanups

* Removing unused includes
* Use member initialization
* Use range-based for loops
* Use nullptr
* Removed no longer used aliases
* Use override
* Don't use else after return
* Use '= delete' to remove implicit members
* Use std::string::empty instead of comparing to ""

1 files changed, 8 insertions, 8 deletions

diff --git a/src/playerinfo.cpp b/src/playerinfo.cpp
index 708dfc9a..a937d71e 100644
--- a/src/playerinfo.cpp
+++ b/src/playerinfo.cpp
@@ -81,8 +81,8 @@ int getAttribute(int id)
     IntMap::const_iterator it = mData.mAttributes.find(id);
     if (it != mData.mAttributes.end())
         return it->second;
-    else
-        return 0;
+
+    return 0;
 }
 
 void setAttribute(int id, int value, bool notify)
@@ -100,8 +100,8 @@ int getStatBase(int id)
     StatMap::const_iterator it = mData.mStats.find(id);
     if (it != mData.mStats.end())
         return it->second.base;
-    else
-        return 0;
+
+    return 0;
 }
 
 void setStatBase(int id, int value, bool notify)
@@ -117,8 +117,8 @@ int getStatMod(int id)
     StatMap::const_iterator it = mData.mStats.find(id);
     if (it != mData.mStats.end())
         return it->second.mod;
-    else
-        return 0;
+
+    return 0;
 }
 
 void setStatMod(int id, int value, bool notify)
@@ -134,8 +134,8 @@ int getStatEffective(int id)
     StatMap::const_iterator it = mData.mStats.find(id);
     if (it != mData.mStats.end())
         return it->second.base + it->second.mod;
-    else
-        return 0;
+
+    return 0;
 }
 
 std::pair<int, int> getStatExperience(int id)