diff options
| author | Thorbjørn Lindeijer <bjorn@lindeijer.nl> | 2024-02-22 13:02:26 +0100 |
|---|---|---|
| committer | Thorbjørn Lindeijer <bjorn@lindeijer.nl> | 2024-02-22 14:32:45 +0100 |
| commit | 81e4f170d8ba4ccbcfa1e6c07bd0522dfc3b6e08 (patch) | |
| tree | e2619b16a5331e5760d94be389d0a3a01427293f /src/avatar.h | |
| parent | d047db79f7034e0e75a85a656d18f40716d197b9 (diff) | |
| download | mana-81e4f170d8ba4ccbcfa1e6c07bd0522dfc3b6e08.tar.gz mana-81e4f170d8ba4ccbcfa1e6c07bd0522dfc3b6e08.tar.bz2 mana-81e4f170d8ba4ccbcfa1e6c07bd0522dfc3b6e08.tar.xz mana-81e4f170d8ba4ccbcfa1e6c07bd0522dfc3b6e08.zip | |
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
Diffstat (limited to 'src/avatar.h')
| -rw-r--r-- | src/avatar.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/avatar.h b/src/avatar.h index bbae9044..29fc8c56 100644 --- a/src/avatar.h +++ b/src/avatar.h @@ -27,12 +27,12 @@ class Avatar { public: - Avatar(const std::string &name = ""); + Avatar(const std::string &name = std::string()); /** * Returns the avatar's name. */ - std::string getName() const { return mName; } + const std::string &getName() const { return mName; } /** * Set the avatar's name. |