From 596484da4fcd301e597a851a9a54b6dc276a286a Mon Sep 17 00:00:00 2001 From: Loic Blot Date: Wed, 23 Sep 2015 18:53:54 +0200 Subject: 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 --- src/content_sao.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'src/content_sao.cpp') 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); } } -- cgit v1.2.3