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

diff --git a/src/localplayer.cpp b/src/localplayer.cpp
index 934155dd..288dfd33 100644
--- a/src/localplayer.cpp
+++ b/src/localplayer.cpp
@@ -920,7 +920,7 @@ void LocalPlayer::pickedUp(const ItemInfo &itemInfo, int amount,
         if (mMap && config.getBoolValue("showpickupparticle"))
         {
             // Show pickup notification
-            std::string msg = "";
+            std::string msg;
             if (amount > 1)
                 msg = strprintf("%i ", amount);
             msg += itemInfo.getName();