General code cleanups

* Use default member initializers
* Use range-based loops
* Don't use 'else' after 'return'
* Removed some unused includes
* Construct empty strings with std::string() instead of ""
* Clear strings with .clear() instead of assigning ""
* Check whether strings are empty with .empty() instead of comparing to ""
* Removed redundant initializations

1 files changed, 5 insertions, 5 deletions

diff --git a/src/compoundsprite.h b/src/compoundsprite.h
index e5b41973..7c91eb73 100644
--- a/src/compoundsprite.h
+++ b/src/compoundsprite.h
@@ -95,13 +95,13 @@ private:
 
     void redraw() const;
 
-    mutable Image *mImage;
-    mutable Image *mAlphaImage;
+    mutable Image *mImage = nullptr;
+    mutable Image *mAlphaImage = nullptr;
 
-    mutable int mWidth, mHeight;
-    mutable int mOffsetX, mOffsetY;
+    mutable int mWidth = 0, mHeight = 0;
+    mutable int mOffsetX = 0, mOffsetY = 0;
 
-    mutable bool mNeedsRedraw;
+    mutable bool mNeedsRedraw = false;
 
     std::vector<Sprite*> mSprites;
 };