From de16b56073dd2c4926327bd990b008a39018f541 Mon Sep 17 00:00:00 2001 From: Andrei Karas Date: Sat, 13 Sep 2014 20:10:15 +0300 Subject: Split processChat for each packet type. --- src/net/eathena/chathandler.cpp | 24 ++++++++++++++++++------ 1 file changed, 18 insertions(+), 6 deletions(-) (limited to 'src/net/eathena/chathandler.cpp') diff --git a/src/net/eathena/chathandler.cpp b/src/net/eathena/chathandler.cpp index 31a585cf9..6e2ab0fb3 100644 --- a/src/net/eathena/chathandler.cpp +++ b/src/net/eathena/chathandler.cpp @@ -85,10 +85,13 @@ void ChatHandler::handleMessage(Net::MessageIn &msg) break; case SMSG_PLAYER_CHAT: - case SMSG_COLOR_MESSAGE: processChat(msg); break; + case SMSG_COLOR_MESSAGE: + processColorChat(msg); + break; + case SMSG_GM_CHAT: processGmChat(msg); break; @@ -266,14 +269,23 @@ void ChatHandler::createChatRoom(const std::string &title, void ChatHandler::processChat(Net::MessageIn &msg) { BLOCK_START("ChatHandler::processChat") - const bool coloredChat = msg.getId() == SMSG_COLOR_MESSAGE; int chatMsgLength = msg.readInt16("len") - 4; - if (coloredChat) + if (chatMsgLength <= 0) { - msg.readInt32("unused"); - msg.readInt32("chat color"); - chatMsgLength -= 8; + BLOCK_END("ChatHandler::processChat") + return; } + + processChatContinue(msg.readRawString(chatMsgLength, "message")); +} + +void ChatHandler::processColorChat(Net::MessageIn &msg) +{ + BLOCK_START("ChatHandler::processChat") + int chatMsgLength = msg.readInt16("len") - 4; + msg.readInt32("unused"); + msg.readInt32("chat color"); + chatMsgLength -= 8; if (chatMsgLength <= 0) { BLOCK_END("ChatHandler::processChat") -- cgit v1.2.3-60-g2f50