diff options
author | Bjørn Lindeijer <bjorn@lindeijer.nl> | 2005-10-09 12:07:25 +0000 |
---|---|---|
committer | Bjørn Lindeijer <bjorn@lindeijer.nl> | 2005-10-09 12:07:25 +0000 |
commit | 6f706879515b4e37a78994aaae6b0e36d00b5f32 (patch) | |
tree | 33785b722f71a1496fc88e4a619dedbc7f716ea8 /src | |
parent | 8bde9095c5840b8d62ebafe11beaed98877d6ac2 (diff) | |
download | mana-client-6f706879515b4e37a78994aaae6b0e36d00b5f32.tar.gz mana-client-6f706879515b4e37a78994aaae6b0e36d00b5f32.tar.bz2 mana-client-6f706879515b4e37a78994aaae6b0e36d00b5f32.tar.xz mana-client-6f706879515b4e37a78994aaae6b0e36d00b5f32.zip |
Fixed map switch to crash the client.
Diffstat (limited to 'src')
-rw-r--r-- | src/engine.cpp | 28 | ||||
-rw-r--r-- | src/game.cpp | 23 |
2 files changed, 33 insertions, 18 deletions
diff --git a/src/engine.cpp b/src/engine.cpp index b7155ebb..dda6896c 100644 --- a/src/engine.cpp +++ b/src/engine.cpp @@ -190,12 +190,38 @@ Map *Engine::getCurrentMap() void Engine::changeMap(const std::string &mapPath) { + // Clean up floor items + empty_floor_items(); + + // Remove the local player, so it is not deleted + if (player_node != NULL) + { + beings.remove(player_node); + } + + // Delete all beings (except the local player) + std::list<Being*>::iterator i; + for (i = beings.begin(); i != beings.end(); i++) + { + delete (*i); + } + beings.clear(); + + // Attempt to load the new map Map *newMap = MapReader::readMap(mapPath); if (!newMap) { logger->error("Could not find map file"); } + // Re-add the local player node and transfer him to the newly loaded map + if (player_node != NULL) + { + beings.push_back(player_node); + player_node->setMap(newMap); + } + + // Start playing new music file when necessary std::string oldMusic = ""; if (mCurrentMap) { @@ -211,6 +237,8 @@ void Engine::changeMap(const std::string &mapPath) } mCurrentMap = newMap; + + // Notify the minimap about the map change minimap->setMap(mCurrentMap); } diff --git a/src/game.cpp b/src/game.cpp index d85149fe..e8692fd9 100644 --- a/src/game.cpp +++ b/src/game.cpp @@ -1472,35 +1472,22 @@ void do_parse() { // Set new map path map_path = "maps/" + msg.readString(16); - map_path= map_path.substr(0, map_path.rfind(".")) + ".tmx.gz"; + map_path = map_path.substr(0, map_path.rfind(".")) + + ".tmx.gz"; Uint16 x = msg.readShort(); Uint16 y = msg.readShort(); - logger->log("Warping to %s (%d, %d)", map_path.c_str(), x, y); + logger->log("Warping to %s (%d, %d)", + map_path.c_str(), x, y); + // Switch the actual map, deleting the previous one engine->changeMap(map_path); tiledMap = engine->getCurrentMap(); - empty_floor_items(); - - // Remove the player, so it is not deleted - beings.remove(player_node); - - // Delete all beings except the local player - std::list<Being *>::iterator i; - for (i = beings.begin(); i != beings.end(); i++) - { - delete (*i); - } - beings.clear(); - autoTarget = NULL; current_npc = 0; - // Re-add the local player node - beings.push_back(player_node); - player_node->action = Being::STAND; player_node->mFrame = 0; player_node->x = x; |