diff options
author | Loic Blot <loic.blot@unix-experience.fr> | 2015-09-23 18:53:54 +0200 |
---|---|---|
committer | est31 <MTest31@outlook.com> | 2015-09-23 19:37:05 +0200 |
commit | 596484da4fcd301e597a851a9a54b6dc276a286a (patch) | |
tree | 500b4b1c05e38357d583c33ac3910d94ce35bfec /src/content_sao.cpp | |
parent | a56aedb4eae7d21864ab8dc56a82f644c4639f7a (diff) | |
download | minetest-596484da4fcd301e597a851a9a54b6dc276a286a.tar.gz minetest-596484da4fcd301e597a851a9a54b6dc276a286a.tar.bz2 minetest-596484da4fcd301e597a851a9a54b6dc276a286a.zip |
Save and remove player by pointer
Why doing things simple ? Use pointer instead of strings to save players and remove them.
Saving players by name does a lookup to find pointer we already have ! Idem with removePlayer
Also remove unused removePlayer(peer_id), it's never called
Diffstat (limited to 'src/content_sao.cpp')
-rw-r--r-- | src/content_sao.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/content_sao.cpp b/src/content_sao.cpp index add1726fc..fe46f3acb 100644 --- a/src/content_sao.cpp +++ b/src/content_sao.cpp @@ -821,8 +821,8 @@ void PlayerSAO::removingFromEnvironment() { m_player->setPlayerSAO(NULL); m_player->peer_id = 0; - m_env->savePlayer(m_player->getName()); - m_env->removePlayer(m_player->getName()); + m_env->savePlayer((RemotePlayer*)m_player); + m_env->removePlayer(m_player); } } |