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/gui/beingpopup.cpp | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) (limited to 'src/gui/beingpopup.cpp') diff --git a/src/gui/beingpopup.cpp b/src/gui/beingpopup.cpp index 56faed39..9cb585db 100644 --- a/src/gui/beingpopup.cpp +++ b/src/gui/beingpopup.cpp @@ -53,9 +53,7 @@ BeingPopup::BeingPopup(): add(mBeingParty); } -BeingPopup::~BeingPopup() -{ -} +BeingPopup::~BeingPopup() = default; void BeingPopup::show(int x, int y, Being *b) { @@ -84,7 +82,7 @@ void BeingPopup::show(int x, int y, Being *b) } else { - mBeingParty->setCaption(""); + mBeingParty->setCaption(std::string()); setContentSize(minWidth + 10, height + 10); } -- cgit v1.2.3-70-g09d2