From 81e4f170d8ba4ccbcfa1e6c07bd0522dfc3b6e08 Mon Sep 17 00:00:00 2001
From: Thorbjørn Lindeijer <bjorn@lindeijer.nl>
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/skilldialog.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'src/gui/skilldialog.cpp')

diff --git a/src/gui/skilldialog.cpp b/src/gui/skilldialog.cpp
index 6ec7f011..96a57e54 100644
--- a/src/gui/skilldialog.cpp
+++ b/src/gui/skilldialog.cpp
@@ -333,7 +333,7 @@ void SkillDialog::loadSkills()
             auto *skill = new SkillInfo;
             skill->id = 1;
             skill->name = "basic";
-            skill->setIcon("");
+            skill->setIcon(std::string());
             skill->modifiable = true;
             skill->visible = true;
             skill->model = model;
-- 
cgit v1.2.3-70-g09d2