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

diff --git a/src/compoundsprite.h b/src/compoundsprite.h
index 7c91eb73..a309ad3d 100644
--- a/src/compoundsprite.h
+++ b/src/compoundsprite.h
@@ -90,9 +90,6 @@ public:
     { mNeedsRedraw = true; }
 
 private:
-    using SpriteIterator = std::vector<Sprite *>::iterator;
-    using SpriteConstIterator = std::vector<Sprite *>::const_iterator;
-
     void redraw() const;
 
     mutable Image *mImage = nullptr;