From 36ba43d6ea38062b17f7e63ef659962bfc51c64d Mon Sep 17 00:00:00 2001 From: Andrei Karas Date: Tue, 6 Jun 2017 23:34:34 +0300 Subject: Fix clang-tidy check readability-implicit-bool-cast. --- src/utils/chatutils.cpp | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) (limited to 'src/utils/chatutils.cpp') diff --git a/src/utils/chatutils.cpp b/src/utils/chatutils.cpp index 456fc4233..53bb689a4 100644 --- a/src/utils/chatutils.cpp +++ b/src/utils/chatutils.cpp @@ -46,10 +46,10 @@ void outStringNormal(ChatTab *const tab, const std::string &str, const std::string &def) { - if (!localPlayer) + if (localPlayer == nullptr) return; - if (!tab) + if (tab == nullptr) { chatHandler->talk(str, GENERAL_CHANNEL); return; @@ -65,7 +65,7 @@ void outStringNormal(ChatTab *const tab, case ChatTabType::GUILD: { const Guild *const guild = localPlayer->getGuild(); - if (guild) + if (guild != nullptr) { #ifdef TMWA_SUPPORT if (guild->getServerGuild()) @@ -74,7 +74,7 @@ void outStringNormal(ChatTab *const tab, return; guildHandler->chat(str); } - else if (guildManager) + else if (guildManager != nullptr) { guildManager->chat(str); } @@ -111,18 +111,18 @@ void outStringNormal(ChatTab *const tab, void replaceVars(std::string &str) { - if (!localPlayer || !actorManager) + if ((localPlayer == nullptr) || (actorManager == nullptr)) return; if (str.find("") != std::string::npos) { const Being *target = localPlayer->getTarget(); - if (!target || target->getType() != ActorType::Player) + if ((target == nullptr) || target->getType() != ActorType::Player) { target = actorManager->findNearestLivingBeing( localPlayer, 20, ActorType::Player, AllowSort_true); } - if (target) + if (target != nullptr) replaceAll(str, "", target->getName()); else replaceAll(str, "", ""); @@ -130,12 +130,12 @@ void replaceVars(std::string &str) if (str.find("") != std::string::npos) { const Being *target = localPlayer->getTarget(); - if (!target || target->getType() != ActorType::Monster) + if ((target == nullptr) || target->getType() != ActorType::Monster) { target = actorManager->findNearestLivingBeing( localPlayer, 20, ActorType::Monster, AllowSort_true); } - if (target) + if (target != nullptr) replaceAll(str, "", target->getName()); else replaceAll(str, "", ""); @@ -167,7 +167,7 @@ void replaceVars(std::string &str) std::string newStr; const Party *party = nullptr; if (localPlayer->isInParty() && - (party = localPlayer->getParty())) + ((party = localPlayer->getParty()) != nullptr)) { party->getNames(names); FOR_EACH (StringVectCIter, it, names) -- cgit v1.2.3-70-g09d2