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

diff --git a/src/item.h b/src/item.h
index 5b64b017..2be75164 100644
--- a/src/item.h
+++ b/src/item.h
@@ -120,13 +120,13 @@ class Item
         const ItemInfo &getInfo() const { return itemDb->get(mId); }
 
     protected:
-        int mId;              /**< Item type id. */
-        Image *mImage;        /**< Item image. */
-        Image *mDrawImage;    /**< Draw image. */
-        int mQuantity;        /**< Number of items. */
-        bool mEquipped;       /**< Item is equipped. */
-        bool mInEquipment;    /**< Item is in equipment */
-        int mInvIndex;        /**< Inventory index. */
+        int mId;                        /**< Item type id. */
+        Image *mImage = nullptr;        /**< Item image. */
+        Image *mDrawImage = nullptr;    /**< Draw image. */
+        int mQuantity;                  /**< Number of items. */
+        bool mEquipped;                 /**< Item is equipped. */
+        bool mInEquipment = false;      /**< Item is in equipment */
+        int mInvIndex;                  /**< Inventory index. */
 };
 
 #endif // ITEM_H