General code cleanups

* Use default member initializers
* Use range-based for loops
* Avoid needless pointer references for ShopItem::mDuplicates
* Removed type aliases that are only used once or twice
* Removed more unused includes
* Removed some unused functions
* Removed superfluous .c_str()
* Rely on default copy and assignment operators for Vector class
* Use std::unique_ptr in some places
* Removed duplicated mPlayerMoney updating in SellDialog
* Removed duplicated Game::handleInput call
* Removed unused SDLInput::mMouseInWindow
* Removed remnant of manual widget positioning in HelpWindow
* Removed superfluous initialization of static pointers

1 files changed, 2 insertions, 5 deletions

diff --git a/src/playerinfo.h b/src/playerinfo.h
index 925c5078..cbb6c748 100644
--- a/src/playerinfo.h
+++ b/src/playerinfo.h
@@ -49,16 +49,13 @@ struct Stat
     int expNeed;
 };
 
-using IntMap = std::map<int, int>;
-using StatMap = std::map<int, Stat>;
-
 /**
  * Backend for core player information.
  */
 struct PlayerInfoBackend
 {
-    IntMap mAttributes;
-    StatMap mStats;
+    std::map<int, int> mAttributes;
+    std::map<int, Stat> mStats;
 };
 
 class Equipment;