diff options
author | Andrei Karas <akaras@inbox.ru> | 2016-08-21 22:52:54 +0300 |
---|---|---|
committer | Andrei Karas <akaras@inbox.ru> | 2016-08-21 22:52:54 +0300 |
commit | 7f72738f54abef2ec0ae080b58bbfde1c3c02c36 (patch) | |
tree | e2ff03a2e380b223f1c592fde0522934c6981171 /src/net | |
parent | 194cda54f0bb15befcb0449152957b16f922bb49 (diff) | |
download | manaplus-7f72738f54abef2ec0ae080b58bbfde1c3c02c36.tar.gz manaplus-7f72738f54abef2ec0ae080b58bbfde1c3c02c36.tar.bz2 manaplus-7f72738f54abef2ec0ae080b58bbfde1c3c02c36.tar.xz manaplus-7f72738f54abef2ec0ae080b58bbfde1c3c02c36.zip |
Improve move3 packet handling for local player and for empty path.
Diffstat (limited to 'src/net')
-rw-r--r-- | src/net/ea/beingrecv.cpp | 15 |
1 files changed, 7 insertions, 8 deletions
diff --git a/src/net/ea/beingrecv.cpp b/src/net/ea/beingrecv.cpp index 9b12583a8..bc1229b5d 100644 --- a/src/net/ea/beingrecv.cpp +++ b/src/net/ea/beingrecv.cpp @@ -422,7 +422,7 @@ void BeingRecv::processBeingMove3(Net::MessageIn &msg) const int len = msg.readInt16("len") - 14; Being *const dstBeing = actorManager->findBeing( msg.readBeingId("being id")); - if (!dstBeing) + if (!dstBeing || dstBeing == localPlayer) { DEBUGLOGSTR("invisible player?"); msg.readInt16("speed"); @@ -436,14 +436,8 @@ void BeingRecv::processBeingMove3(Net::MessageIn &msg) dstBeing->setWalkSpeed(speed); const int16_t x = msg.readInt16("x"); const int16_t y = msg.readInt16("y"); - - dstBeing->setAction(BeingAction::STAND, 0); - dstBeing->setTileCoords(x, y); - - if (dstBeing == localPlayer) - return; - const unsigned char *moves = msg.readBytes(len, "moving path"); + Path path; if (moves) { @@ -486,6 +480,11 @@ void BeingRecv::processBeingMove3(Net::MessageIn &msg) delete [] moves; } + if (path.empty()) + return; + + dstBeing->setAction(BeingAction::STAND, 0); + dstBeing->setTileCoords(x, y); dstBeing->setPath(path); BLOCK_END("BeingRecv::processBeingMove3") } |