summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAndrei Karas <akaras@inbox.ru>2013-02-22 23:30:25 +0300
committerAndrei Karas <akaras@inbox.ru>2013-02-22 23:30:25 +0300
commita3ecadac89eafbacd0c6b3764d5977bcac242c57 (patch)
tree43306287103833ff3c0816338283e69c4ee3d695
parent24e0a7941965b6130fa008c0783d559c57c91005 (diff)
downloadplus-a3ecadac89eafbacd0c6b3764d5977bcac242c57.tar.gz
plus-a3ecadac89eafbacd0c6b3764d5977bcac242c57.tar.bz2
plus-a3ecadac89eafbacd0c6b3764d5977bcac242c57.tar.xz
plus-a3ecadac89eafbacd0c6b3764d5977bcac242c57.zip
const and other minor changes in commands.cpp
-rw-r--r--src/commands.cpp87
-rw-r--r--src/net/eathena/tradehandler.cpp2
-rw-r--r--src/net/eathena/tradehandler.h2
-rw-r--r--src/net/tmwa/tradehandler.cpp2
-rw-r--r--src/net/tmwa/tradehandler.h2
-rw-r--r--src/net/tradehandler.h2
6 files changed, 44 insertions, 53 deletions
diff --git a/src/commands.cpp b/src/commands.cpp
index 25835ace9..e4247bd0e 100644
--- a/src/commands.cpp
+++ b/src/commands.cpp
@@ -123,6 +123,9 @@ static void changeRelation(const std::string &args,
const std::string &relationText,
ChatTab *const tab)
{
+ if (!tab)
+ return;
+
if (args.empty())
{
if (tab)
@@ -132,11 +135,8 @@ static void changeRelation(const std::string &args,
if (player_relations.getRelation(args) == relation)
{
- if (tab)
- {
- tab->chatLog(strprintf(_("Player already %s!"),
- relationText.c_str()), BY_SERVER);
- }
+ tab->chatLog(strprintf(_("Player already %s!"),
+ relationText.c_str()), BY_SERVER);
return;
}
else
@@ -146,19 +146,13 @@ static void changeRelation(const std::string &args,
if (player_relations.getRelation(args) == relation)
{
- if (tab)
- {
- tab->chatLog(strprintf(_("Player successfully %s!"),
- relationText.c_str()), BY_SERVER);
- }
+ tab->chatLog(strprintf(_("Player successfully %s!"),
+ relationText.c_str()), BY_SERVER);
}
else
{
- if (tab)
- {
- tab->chatLog(strprintf(_("Player could not be %s!"),
- relationText.c_str()), BY_SERVER);
- }
+ tab->chatLog(strprintf(_("Player could not be %s!"),
+ relationText.c_str()), BY_SERVER);
}
}
@@ -200,8 +194,6 @@ static void outStringNormal(ChatTab *const tab,
}
case ChatTab::TAB_GUILD:
{
- if (!player_node)
- return;
const Guild *const guild = player_node->getGuild();
if (guild)
{
@@ -279,8 +271,8 @@ impHandler0(who)
impHandler(msg)
{
- std::string recvnick("");
- std::string msg("");
+ std::string recvnick;
+ std::string msg;
if (args.substr(0, 1) == "\"")
{
@@ -338,9 +330,12 @@ impHandler(query)
}
}
- tab->chatLog(strprintf(_("Cannot create a whisper tab for nick \"%s\"! "
- "It either already exists, or is you."),
- args.c_str()), BY_SERVER);
+ if (tab)
+ {
+ tab->chatLog(strprintf(_("Cannot create a whisper tab for nick \"%s\"! "
+ "It either already exists, or is you."),
+ args.c_str()), BY_SERVER);
+ }
}
impHandler0(clear)
@@ -474,8 +469,8 @@ impHandler(unignore)
return;
}
- if (player_relations.getRelation(args) != PlayerRelation::NEUTRAL
- && player_relations.getRelation(args) != PlayerRelation::FRIEND)
+ const PlayerRelation::Relation rel = player_relations.getRelation(args);
+ if (rel != PlayerRelation::NEUTRAL && rel != PlayerRelation::FRIEND)
{
player_relations.setRelation(args, PlayerRelation::NEUTRAL);
}
@@ -662,13 +657,9 @@ impHandler(follow)
return;
if (!args.empty())
- {
player_node->setFollow(args);
- }
else if (tab && tab->getType() == ChatTab::TAB_WHISPER)
- {
player_node->setFollow(static_cast<WhisperTab*>(tab)->getNick());
- }
}
impHandler(imitation)
@@ -750,7 +741,7 @@ impHandler1(trade)
if (!actorSpriteManager)
return;
- Being *const being = actorSpriteManager->findBeingByName(
+ const Being *const being = actorSpriteManager->findBeingByName(
args, Being::PLAYER);
if (being)
{
@@ -778,12 +769,13 @@ impHandler0(dirs)
impHandler2(info)
{
+ if (!tab || !player_node)
+ return;
+
switch (tab->getType())
{
case ChatTab::TAB_GUILD:
{
- if (!player_node)
- return;
const Guild *const guild = player_node->getGuild();
if (guild)
Net::getGuildHandler()->info(guild->getId());
@@ -930,9 +922,9 @@ impHandler0(cacheInfo)
{
if (!cache[f].empty())
{
- all += static_cast<int>(cache[f].size());
- str += strprintf("%d: %u, ", f,
- static_cast<unsigned int>(cache[f].size()));
+ const int sz = static_cast<int>(cache[f].size());
+ all += sz;
+ str += strprintf("%d: %u, ", f, sz);
}
}
debugChatTab->chatLog(str);
@@ -958,8 +950,7 @@ impHandler0(serverUnIgnoreAll)
impHandler2(dumpGraphics)
{
- std::string str;
- str = strprintf ("%s,%s,%dX%dX%d,", PACKAGE_OS, SMALL_VERSION,
+ std::string str = strprintf ("%s,%s,%dX%dX%d,", PACKAGE_OS, SMALL_VERSION,
mainGraphics->getWidth(), mainGraphics->getHeight(),
mainGraphics->getBpp());
@@ -1000,17 +991,17 @@ impHandler2(dumpGraphics)
impHandler0(dumpEnvironment)
{
- logger->log("Start environment variables");
+ logger->log1("Start environment variables");
for (char **env = environ; *env; ++ env)
logger->log1(*env);
- logger->log("End environment variables");
+ logger->log1("End environment variables");
if (debugChatTab)
debugChatTab->chatLog(_("Environment variables dumped"));
}
impHandler2(dumpTests)
{
- std::string str = config.getStringValue("testInfo");
+ const std::string str = config.getStringValue("testInfo");
outStringNormal(tab, str, str);
}
@@ -1022,7 +1013,7 @@ impHandler1(setDrop)
impHandler0(error)
{
- int *ptr = nullptr;
+ const int *const ptr = nullptr;
logger->log("test %d", *ptr);
}
@@ -1058,15 +1049,15 @@ void showRes(std::string str, ResourceManager::Resources *res)
debugChatTab->chatLog(str + toString(res->size()));
logger->log(str + toString(res->size()));
ResourceManager::ResourceIterator iter = res->begin();
- ResourceManager::ResourceIterator iter_end = res->end();
+ const ResourceManager::ResourceIterator iter_end = res->end();
while (iter != iter_end)
{
if (iter->second && iter->second->getRefCount())
{
- std::string type = " ";
- std::string isNew = "N";
+ char type = ' ';
+ char isNew = 'N';
if (iter->second->getDumped())
- isNew = "O";
+ isNew = 'O';
else
iter->second->setDumped(true);
@@ -1075,12 +1066,12 @@ void showRes(std::string str, ResourceManager::Resources *res)
Image *const image = dynamic_cast<Image *const>(iter->second);
int id = 0;
if (subImage)
- type = "S";
+ type = 'S';
else if (image)
- type = "I";
+ type = 'I';
if (image)
id = image->getGLImage();
- logger->log("Resource %s%s: %s (%d) id=%d", type.c_str(),
+ logger->log("Resource %c%c: %s (%d) id=%d", type.c_str(),
isNew.c_str(), iter->second->getIdPath().c_str(),
iter->second->getRefCount(), id);
}
@@ -1093,7 +1084,7 @@ impHandler1(dump)
if (!debugChatTab)
return;
- ResourceManager *resman = ResourceManager::getInstance();
+ ResourceManager *const resman = ResourceManager::getInstance();
if (!args.empty())
{
diff --git a/src/net/eathena/tradehandler.cpp b/src/net/eathena/tradehandler.cpp
index 9ed75416e..6db01aa67 100644
--- a/src/net/eathena/tradehandler.cpp
+++ b/src/net/eathena/tradehandler.cpp
@@ -93,7 +93,7 @@ void TradeHandler::handleMessage(Net::MessageIn &msg)
}
}
-void TradeHandler::request(Being *being)
+void TradeHandler::request(const Being *const being)
{
if (!being)
return;
diff --git a/src/net/eathena/tradehandler.h b/src/net/eathena/tradehandler.h
index 395f9337a..0cd826ca2 100644
--- a/src/net/eathena/tradehandler.h
+++ b/src/net/eathena/tradehandler.h
@@ -42,7 +42,7 @@ class TradeHandler final : public MessageHandler, public Ea::TradeHandler
void handleMessage(Net::MessageIn &msg);
- void request(Being *being);
+ void request(const Being *const being);
void respond(bool accept);
diff --git a/src/net/tmwa/tradehandler.cpp b/src/net/tmwa/tradehandler.cpp
index 12ff23daf..a05f28401 100644
--- a/src/net/tmwa/tradehandler.cpp
+++ b/src/net/tmwa/tradehandler.cpp
@@ -95,7 +95,7 @@ void TradeHandler::handleMessage(Net::MessageIn &msg)
BLOCK_END("TradeHandler::handleMessage")
}
-void TradeHandler::request(Being *being)
+void TradeHandler::request(const Being *const being)
{
if (!being)
return;
diff --git a/src/net/tmwa/tradehandler.h b/src/net/tmwa/tradehandler.h
index d96d325a6..dfa5840b0 100644
--- a/src/net/tmwa/tradehandler.h
+++ b/src/net/tmwa/tradehandler.h
@@ -42,7 +42,7 @@ class TradeHandler final : public MessageHandler, public Ea::TradeHandler
void handleMessage(Net::MessageIn &msg);
- void request(Being *being);
+ void request(const Being *const being);
void respond(bool accept);
diff --git a/src/net/tradehandler.h b/src/net/tradehandler.h
index e7cfc2925..2eeaaf635 100644
--- a/src/net/tradehandler.h
+++ b/src/net/tradehandler.h
@@ -37,7 +37,7 @@ class TradeHandler
virtual ~TradeHandler()
{ }
- virtual void request(Being *being A_UNUSED)
+ virtual void request(const Being *const being A_UNUSED)
{ }
virtual void respond(bool accept A_UNUSED)