From 81e4f170d8ba4ccbcfa1e6c07bd0522dfc3b6e08 Mon Sep 17 00:00:00 2001 From: Thorbjørn Lindeijer Date: Thu, 22 Feb 2024 13:02:26 +0100 Subject: 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 --- src/localplayer.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src/localplayer.cpp') 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(); -- cgit v1.2.3-70-g09d2