diff options
-rw-r--r-- | src/CMakeLists.txt | 4 | ||||
-rw-r--r-- | src/Makefile.am | 6 | ||||
-rw-r--r-- | src/actormanager.cpp | 5 | ||||
-rw-r--r-- | src/gui/fonts/textchunksmall.cpp | 3 | ||||
-rw-r--r-- | src/gui/fonts/textchunksmall.h | 3 | ||||
-rw-r--r-- | src/gui/windows/inventorywindow.cpp | 7 | ||||
-rw-r--r-- | src/net/serverinfo.h | 3 | ||||
-rw-r--r-- | src/net/tmwa/buyingstorehandler.cpp | 3 | ||||
-rw-r--r-- | src/utils/parameters.cpp (renamed from src/utils/paramerers.cpp) | 9 | ||||
-rw-r--r-- | src/utils/parameters.h (renamed from src/utils/paramerers.h) | 0 | ||||
-rw-r--r-- | src/utils/parameters_unittest.cc (renamed from src/utils/paramerers_unittest.cc) | 2 | ||||
-rw-r--r-- | src/vector.h | 3 |
12 files changed, 28 insertions, 20 deletions
diff --git a/src/CMakeLists.txt b/src/CMakeLists.txt index fe622c0c0..22d0cbdbc 100644 --- a/src/CMakeLists.txt +++ b/src/CMakeLists.txt @@ -790,8 +790,8 @@ SET(SRCS utils/langs.cpp utils/langs.h utils/mathutils.h - utils/paramerers.cpp - utils/paramerers.h + utils/parameters.cpp + utils/parameters.h utils/paths.cpp utils/paths.h utils/perfomance.cpp diff --git a/src/Makefile.am b/src/Makefile.am index 756e695a8..a3a47de58 100644 --- a/src/Makefile.am +++ b/src/Makefile.am @@ -441,8 +441,8 @@ SRC += events/actionevent.h \ utils/mathutils.h \ utils/mkdir.cpp \ utils/mkdir.h \ - utils/paramerers.cpp \ - utils/paramerers.h \ + utils/parameters.cpp \ + utils/parameters.h \ utils/paths.cpp \ utils/paths.h \ utils/perfomance.cpp \ @@ -1732,7 +1732,7 @@ manaplustests_SOURCES = ${manaplus_SOURCES} \ utils/mathutils_unittest.cc \ utils/files_unittest.cc \ utils/stringutils_unittest.cc \ - utils/paramerers_unittest.cc \ + utils/parameters_unittest.cc \ resources/mstack_unittest.cc \ utils/translation/poparser_unittest.cc \ resources/sprite/animatedsprite_unittest.cc \ diff --git a/src/actormanager.cpp b/src/actormanager.cpp index d851864a3..7755edbc3 100644 --- a/src/actormanager.cpp +++ b/src/actormanager.cpp @@ -1610,8 +1610,11 @@ void ActorManager::getMobNames(StringVect &names) const } const Being *const being = static_cast<const Being*>(*it); - if (being->getType() == ActorType::Monster && !being->getName().empty()) + if (being->getType() == ActorType::Monster && + !being->getName().empty()) + { names.push_back(being->getName()); + } } } diff --git a/src/gui/fonts/textchunksmall.cpp b/src/gui/fonts/textchunksmall.cpp index f8c84fdb8..a8012522e 100644 --- a/src/gui/fonts/textchunksmall.cpp +++ b/src/gui/fonts/textchunksmall.cpp @@ -40,11 +40,12 @@ TextChunkSmall::TextChunkSmall(const TextChunkSmall &old) : { } -void TextChunkSmall::operator=(const TextChunkSmall &chunk) +TextChunkSmall &TextChunkSmall::operator=(const TextChunkSmall &chunk) { text = chunk.text; color = chunk.color; color2 = chunk.color2; + return *this; } bool TextChunkSmall::operator==(const TextChunkSmall &chunk) const diff --git a/src/gui/fonts/textchunksmall.h b/src/gui/fonts/textchunksmall.h index 75c4e6dd7..64a265da4 100644 --- a/src/gui/fonts/textchunksmall.h +++ b/src/gui/fonts/textchunksmall.h @@ -38,7 +38,8 @@ class TextChunkSmall final bool operator==(const TextChunkSmall &restrict chunk) const restrict2; bool operator<(const TextChunkSmall &restrict chunk) const restrict2; - void operator=(const TextChunkSmall &restrict chunk) restrict2; + TextChunkSmall &operator=(const TextChunkSmall &restrict chunk) + restrict2; std::string text; Color color; Color color2; diff --git a/src/gui/windows/inventorywindow.cpp b/src/gui/windows/inventorywindow.cpp index 5c6ae3faa..881e10c1a 100644 --- a/src/gui/windows/inventorywindow.cpp +++ b/src/gui/windows/inventorywindow.cpp @@ -588,7 +588,12 @@ void InventoryWindow::mouseClicked(MouseEvent &event) const int my = event.getY() + getY(); if (popupMenu) - popupMenu->showPopup(this, mx, my, item, mInventory->getType()); + { + popupMenu->showPopup(this, + mx, my, + item, + mInventory->getType()); + } } } else diff --git a/src/net/serverinfo.h b/src/net/serverinfo.h index 7df9e0066..efbdf015a 100644 --- a/src/net/serverinfo.h +++ b/src/net/serverinfo.h @@ -89,7 +89,7 @@ class ServerInfo final version.second = info.version.second; } - void operator=(const ServerInfo &info) + ServerInfo &operator=(const ServerInfo &info) { type = info.type; name = info.name; @@ -105,6 +105,7 @@ class ServerInfo final persistentIp = info.persistentIp; version.first = info.version.first; version.second = info.version.second; + return *this; } bool isValid() const A_WARN_UNUSED diff --git a/src/net/tmwa/buyingstorehandler.cpp b/src/net/tmwa/buyingstorehandler.cpp index 882206761..c31bf5989 100644 --- a/src/net/tmwa/buyingstorehandler.cpp +++ b/src/net/tmwa/buyingstorehandler.cpp @@ -35,7 +35,8 @@ BuyingStoreHandler::BuyingStoreHandler() void BuyingStoreHandler::create(const std::string &name A_UNUSED, const int maxMoney A_UNUSED, const bool flag A_UNUSED, - const std::vector<ShopItem*> &items A_UNUSED) const + const std::vector<ShopItem*> &items A_UNUSED) + const { } diff --git a/src/utils/paramerers.cpp b/src/utils/parameters.cpp index aa3ae350f..467c31553 100644 --- a/src/utils/paramerers.cpp +++ b/src/utils/parameters.cpp @@ -18,14 +18,10 @@ * along with this program. If not, see <http://www.gnu.org/licenses/>. */ -#include "utils/paramerers.h" - -#include "logger.h" +#include "utils/parameters.h" #include "utils/stringutils.h" -#include <sstream> - #include "debug.h" static inline void addToken(StringVect &tokens, @@ -94,7 +90,7 @@ static inline size_t findNextSplit(const std::string &str, idx2 = findNextQuote(str, quote, idx2 + 1); if (idx2 == std::string::npos) return std::string::npos; // no close quote, here error - // searching next + // set position for next separator or quote pos = idx2 + 1; } else @@ -102,7 +98,6 @@ static inline size_t findNextSplit(const std::string &str, return idx1; } } - return idx1; } bool splitParameters(StringVect &tokens, diff --git a/src/utils/paramerers.h b/src/utils/parameters.h index d93f7a207..d93f7a207 100644 --- a/src/utils/paramerers.h +++ b/src/utils/parameters.h diff --git a/src/utils/paramerers_unittest.cc b/src/utils/parameters_unittest.cc index fbccef0e1..f1c4557ee 100644 --- a/src/utils/paramerers_unittest.cc +++ b/src/utils/parameters_unittest.cc @@ -20,7 +20,7 @@ #include "catch.hpp" -#include "utils/paramerers.h" +#include "utils/parameters.h" #include "debug.h" diff --git a/src/vector.h b/src/vector.h index 9f3ca47c0..d24e91e9b 100644 --- a/src/vector.h +++ b/src/vector.h @@ -72,11 +72,12 @@ class Vector final return x == 0.0F && y == 0.0F && z == 0.0F; } - void operator=(const Vector &v) + Vector &operator=(const Vector &v) { x = v.x; y = v.y; z = v.z; + return *this; } /** |