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

diff --git a/src/inventory.h b/src/inventory.h
index b4253b72..891a0d7b 100644
--- a/src/inventory.h
+++ b/src/inventory.h
@@ -30,7 +30,7 @@ class Item;
 class InventoryListener
 {
 public:
-    virtual ~InventoryListener() {}
+    virtual ~InventoryListener() = default;
 
     virtual void slotsChanged(Inventory* inventory) = 0;
 
@@ -135,8 +135,7 @@ class Inventory
         { return mType == INVENTORY; }
 
     protected:
-        using InventoryListenerList = std::list<InventoryListener *>;
-        InventoryListenerList mInventoryListeners;
+        std::list<InventoryListener *> mInventoryListeners;
 
         void distributeSlotsChangedEvent();