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

diff --git a/src/particlecontainer.h b/src/particlecontainer.h
index f01618e1..3905ee49 100644
--- a/src/particlecontainer.h
+++ b/src/particlecontainer.h
@@ -67,7 +67,7 @@ protected:
 /**
  * Linked list of particle effects.
  */
-class ParticleList : public ParticleContainer
+class ParticleList final : public ParticleContainer
 {
 public:
     ParticleList(ParticleContainer *parent = nullptr, bool delParent = true);
@@ -94,7 +94,7 @@ protected:
 /**
  * Particle container with indexing facilities
  */
-class ParticleVector : public ParticleContainer
+class ParticleVector final : public ParticleContainer
 {
 public:
     ParticleVector(ParticleContainer *parent = nullptr, bool delParent = true);
@@ -104,12 +104,12 @@ public:
      * Sets a particle at a specified index.  Kills the previous particle
      * there, if needed.
      */
-    virtual void setLocally(int index, Particle *particle);
+    void setLocally(int index, Particle *particle);
 
     /**
      * Removes a particle at a specified index
      */
-    virtual void delLocally(int index);
+    void delLocally(int index);
 
     void clearLocally() override;
     void moveTo(float x, float y) override;