Don't ship with broken languages (#4940)
[oweals/minetest.git] / src / clientiface.cpp
index 06d87d2a42eecfe818fd0baaf5daae7204756712..0390cf0ffc55e821a9dbf45f5dfe79f8f45ff4a7 100644 (file)
@@ -17,28 +17,57 @@ with this program; if not, write to the Free Software Foundation, Inc.,
 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
 */
 
+#include <sstream>
+
 #include "clientiface.h"
-#include "player.h"
+#include "util/numeric.h"
+#include "util/mathconstants.h"
+#include "remoteplayer.h"
 #include "settings.h"
 #include "mapblock.h"
-#include "connection.h"
+#include "network/connection.h"
 #include "environment.h"
 #include "map.h"
 #include "emerge.h"
-#include "serverobject.h"              // TODO this is used for cleanup of only
-
-#include "util/numeric.h"
-#include "util/mathconstants.h"
+#include "content_sao.h"              // TODO this is used for cleanup of only
+#include "log.h"
+#include "util/srp.h"
+
+const char *ClientInterface::statenames[] = {
+       "Invalid",
+       "Disconnecting",
+       "Denied",
+       "Created",
+       "AwaitingInit2",
+       "HelloSent",
+       "InitDone",
+       "DefinitionsSent",
+       "Active",
+       "SudoMode",
+};
+
+
+
+std::string ClientInterface::state2Name(ClientState state)
+{
+       return statenames[state];
+}
 
-#include "main.h"                      // for g_settings
+void RemoteClient::ResendBlockIfOnWire(v3s16 p)
+{
+       // if this block is on wire, mark it for sending again as soon as possible
+       if (m_blocks_sending.find(p) != m_blocks_sending.end()) {
+               SetBlockNotSent(p);
+       }
+}
 
-void RemoteClient::GetNextBlocks(
+void RemoteClient::GetNextBlocks (
                ServerEnvironment *env,
                EmergeManager * emerge,
                float dtime,
                std::vector<PrioritySortedBlockTransfer> &dest)
 {
-       DSTACK(__FUNCTION_NAME);
+       DSTACK(FUNCTION_NAME);
 
 
        // Increment timers
@@ -48,9 +77,13 @@ void RemoteClient::GetNextBlocks(
        if(m_nothing_to_send_pause_timer >= 0)
                return;
 
-       Player *player = env->getPlayer(peer_id);
+       RemotePlayer *player = env->getPlayer(peer_id);
        // This can happen sometimes; clients and players are not in perfect sync.
-       if(player == NULL)
+       if (player == NULL)
+               return;
+
+       PlayerSAO *sao = player->getPlayerSAO();
+       if (sao == NULL)
                return;
 
        // Won't send anything if already sending
@@ -61,7 +94,7 @@ void RemoteClient::GetNextBlocks(
                return;
        }
 
-       v3f playerpos = player->getPosition();
+       v3f playerpos = sao->getBasePosition();
        v3f playerspeed = player->getSpeed();
        v3f playerspeeddir(0,0,0);
        if(playerspeed.getLength() > 1.0*BS)
@@ -74,10 +107,10 @@ void RemoteClient::GetNextBlocks(
        v3s16 center = getNodeBlockPos(center_nodepos);
 
        // Camera position and direction
-       v3f camera_pos = player->getEyePosition();
+       v3f camera_pos = sao->getEyePosition();
        v3f camera_dir = v3f(0,0,1);
-       camera_dir.rotateYZBy(player->getPitch());
-       camera_dir.rotateXZBy(player->getYaw());
+       camera_dir.rotateYZBy(sao->getPitch());
+       camera_dir.rotateXZBy(sao->getYaw());
 
        /*infostream<<"camera_dir=("<<camera_dir.X<<","<<camera_dir.Y<<","
                        <<camera_dir.Z<<")"<<std::endl;*/
@@ -140,8 +173,20 @@ void RemoteClient::GetNextBlocks(
        */
        s32 new_nearest_unsent_d = -1;
 
-       s16 d_max = g_settings->getS16("max_block_send_distance");
-       s16 d_max_gen = g_settings->getS16("max_block_generate_distance");
+       // get view range and camera fov from the client
+       s16 wanted_range = sao->getWantedRange();
+       float camera_fov = sao->getFov();
+       // if FOV, wanted_range are not available (old client), fall back to old default
+       if (wanted_range <= 0) wanted_range = 1000;
+       if (camera_fov <= 0) camera_fov = (72.0*M_PI/180) * 4./3.;
+
+       const s16 full_d_max = MYMIN(g_settings->getS16("max_block_send_distance"), wanted_range);
+       const s16 d_opt = MYMIN(g_settings->getS16("block_send_optimize_distance"), wanted_range);
+       const s16 d_blocks_in_sight = full_d_max * BS * MAP_BLOCKSIZE;
+       //infostream << "Fov from client " << camera_fov << " full_d_max " << full_d_max << std::endl;
+
+       s16 d_max = full_d_max;
+       s16 d_max_gen = MYMIN(g_settings->getS16("max_block_generate_distance"), wanted_range);
 
        // Don't loop very much at a time
        s16 max_d_increment_at_time = 2;
@@ -151,21 +196,18 @@ void RemoteClient::GetNextBlocks(
        s32 nearest_emerged_d = -1;
        s32 nearest_emergefull_d = -1;
        s32 nearest_sent_d = -1;
-       bool queue_is_full = false;
+       //bool queue_is_full = false;
 
        s16 d;
-       for(d = d_start; d <= d_max; d++)
-       {
+       for(d = d_start; d <= d_max; d++) {
                /*
                        Get the border/face dot coordinates of a "d-radiused"
                        box
                */
-               std::list<v3s16> list;
-               getFacePositions(list, d);
+               std::vector<v3s16> list = FacePositionCache::getFacePositions(d);
 
-               std::list<v3s16>::iterator li;
-               for(li=list.begin(); li!=list.end(); ++li)
-               {
+               std::vector<v3s16>::iterator li;
+               for(li = list.begin(); li != list.end(); ++li) {
                        v3s16 p = *li + center;
 
                        /*
@@ -184,48 +226,31 @@ void RemoteClient::GetNextBlocks(
                                max_simul_dynamic = max_simul_sends_setting;
 
                        // Don't select too many blocks for sending
-                       if(num_blocks_selected >= max_simul_dynamic)
-                       {
-                               queue_is_full = true;
+                       if (num_blocks_selected >= max_simul_dynamic) {
+                               //queue_is_full = true;
                                goto queue_full_break;
                        }
 
                        // Don't send blocks that are currently being transferred
-                       if(m_blocks_sending.find(p) != m_blocks_sending.end())
+                       if (m_blocks_sending.find(p) != m_blocks_sending.end())
                                continue;
 
                        /*
                                Do not go over-limit
                        */
-                       if(p.X < -MAP_GENERATION_LIMIT / MAP_BLOCKSIZE
-                       || p.X > MAP_GENERATION_LIMIT / MAP_BLOCKSIZE
-                       || p.Y < -MAP_GENERATION_LIMIT / MAP_BLOCKSIZE
-                       || p.Y > MAP_GENERATION_LIMIT / MAP_BLOCKSIZE
-                       || p.Z < -MAP_GENERATION_LIMIT / MAP_BLOCKSIZE
-                       || p.Z > MAP_GENERATION_LIMIT / MAP_BLOCKSIZE)
+                       if (blockpos_over_limit(p))
                                continue;
 
                        // If this is true, inexistent block will be made from scratch
                        bool generate = d <= d_max_gen;
 
-                       {
-                               /*// Limit the generating area vertically to 2/3
-                               if(abs(p.Y - center.Y) > d_max_gen - d_max_gen / 3)
-                                       generate = false;*/
-
-                               // Limit the send area vertically to 1/2
-                               if(abs(p.Y - center.Y) > d_max / 2)
-                                       continue;
-                       }
-
                        /*
                                Don't generate or send if not in sight
                                FIXME This only works if the client uses a small enough
                                FOV setting. The default of 72 degrees is fine.
                        */
 
-                       float camera_fov = (72.0*M_PI/180) * 4./3.;
-                       if(isBlockInSight(p, camera_pos, camera_dir, camera_fov, 10000*BS) == false)
+                       if(isBlockInSight(p, camera_pos, camera_dir, camera_fov, d_blocks_in_sight) == false)
                        {
                                continue;
                        }
@@ -275,7 +300,7 @@ void RemoteClient::GetNextBlocks(
                                        Block is near ground level if night-time mesh
                                        differs from day-time mesh.
                                */
-                               if(d >= 4)
+                               if(d >= d_opt)
                                {
                                        if(block->getDayNightDiff() == false)
                                                continue;
@@ -332,7 +357,7 @@ queue_full_break:
        } else if(nearest_emergefull_d != -1){
                new_nearest_unsent_d = nearest_emergefull_d;
        } else {
-               if(d > g_settings->getS16("max_block_send_distance")){
+               if(d > full_d_max){
                        new_nearest_unsent_d = 0;
                        m_nothing_to_send_pause_timer = 2.0;
                } else {
@@ -349,17 +374,21 @@ queue_full_break:
 
 void RemoteClient::GotBlock(v3s16 p)
 {
-       if(m_blocks_sending.find(p) != m_blocks_sending.end())
-               m_blocks_sending.erase(p);
-       else
-       {
-               m_excess_gotblocks++;
+       if (m_blocks_modified.find(p) == m_blocks_modified.end()) {
+               if (m_blocks_sending.find(p) != m_blocks_sending.end())
+                       m_blocks_sending.erase(p);
+               else
+                       m_excess_gotblocks++;
+
+               m_blocks_sent.insert(p);
        }
-       m_blocks_sent.insert(p);
 }
 
 void RemoteClient::SentBlock(v3s16 p)
 {
+       if (m_blocks_modified.find(p) != m_blocks_modified.end())
+               m_blocks_modified.erase(p);
+
        if(m_blocks_sending.find(p) == m_blocks_sending.end())
                m_blocks_sending[p] = 0.0;
        else
@@ -370,22 +399,26 @@ void RemoteClient::SentBlock(v3s16 p)
 void RemoteClient::SetBlockNotSent(v3s16 p)
 {
        m_nearest_unsent_d = 0;
+       m_nothing_to_send_pause_timer = 0;
 
        if(m_blocks_sending.find(p) != m_blocks_sending.end())
                m_blocks_sending.erase(p);
        if(m_blocks_sent.find(p) != m_blocks_sent.end())
                m_blocks_sent.erase(p);
+       m_blocks_modified.insert(p);
 }
 
 void RemoteClient::SetBlocksNotSent(std::map<v3s16, MapBlock*> &blocks)
 {
        m_nearest_unsent_d = 0;
+       m_nothing_to_send_pause_timer = 0;
 
        for(std::map<v3s16, MapBlock*>::iterator
                        i = blocks.begin();
                        i != blocks.end(); ++i)
        {
                v3s16 p = i->first;
+               m_blocks_modified.insert(p);
 
                if(m_blocks_sending.find(p) != m_blocks_sending.end())
                        m_blocks_sending.erase(p);
@@ -396,125 +429,170 @@ void RemoteClient::SetBlocksNotSent(std::map<v3s16, MapBlock*> &blocks)
 
 void RemoteClient::notifyEvent(ClientStateEvent event)
 {
+       std::ostringstream myerror;
        switch (m_state)
        {
-       case Invalid:
-               assert("State update for client in invalid state" != 0);
+       case CS_Invalid:
+               //intentionally do nothing
                break;
-
-       case Created:
-               switch(event)
-               {
-               case Init:
-                       m_state = InitSent;
+       case CS_Created:
+               switch (event) {
+               case CSE_Hello:
+                       m_state = CS_HelloSent;
                        break;
-
-               case Disconnect:
-                       m_state = Disconnecting;
+               case CSE_InitLegacy:
+                       m_state = CS_AwaitingInit2;
                        break;
-
-               case SetDenied:
-                       m_state = Denied;
+               case CSE_Disconnect:
+                       m_state = CS_Disconnecting;
+                       break;
+               case CSE_SetDenied:
+                       m_state = CS_Denied;
                        break;
-
                /* GotInit2 SetDefinitionsSent SetMediaSent */
                default:
-                       assert("Invalid client state transition!" == 0);
+                       myerror << "Created: Invalid client state transition! " << event;
+                       throw ClientStateError(myerror.str());
                }
                break;
-
-       case Denied:
+       case CS_Denied:
                /* don't do anything if in denied state */
                break;
-
-       case InitSent:
+       case CS_HelloSent:
+               switch(event)
+               {
+               case CSE_AuthAccept:
+                       m_state = CS_AwaitingInit2;
+                       if ((chosen_mech == AUTH_MECHANISM_SRP)
+                                       || (chosen_mech == AUTH_MECHANISM_LEGACY_PASSWORD))
+                               srp_verifier_delete((SRPVerifier *) auth_data);
+                       chosen_mech = AUTH_MECHANISM_NONE;
+                       break;
+               case CSE_Disconnect:
+                       m_state = CS_Disconnecting;
+                       break;
+               case CSE_SetDenied:
+                       m_state = CS_Denied;
+                       if ((chosen_mech == AUTH_MECHANISM_SRP)
+                                       || (chosen_mech == AUTH_MECHANISM_LEGACY_PASSWORD))
+                               srp_verifier_delete((SRPVerifier *) auth_data);
+                       chosen_mech = AUTH_MECHANISM_NONE;
+                       break;
+               default:
+                       myerror << "HelloSent: Invalid client state transition! " << event;
+                       throw ClientStateError(myerror.str());
+               }
+               break;
+       case CS_AwaitingInit2:
                switch(event)
                {
-               case GotInit2:
+               case CSE_GotInit2:
                        confirmSerializationVersion();
-                       m_state = InitDone;
+                       m_state = CS_InitDone;
                        break;
-
-               case Disconnect:
-                       m_state = Disconnecting;
+               case CSE_Disconnect:
+                       m_state = CS_Disconnecting;
                        break;
-
-               case SetDenied:
-                       m_state = Denied;
+               case CSE_SetDenied:
+                       m_state = CS_Denied;
                        break;
 
                /* Init SetDefinitionsSent SetMediaSent */
                default:
-                       assert("Invalid client state transition!" == 0);
+                       myerror << "InitSent: Invalid client state transition! " << event;
+                       throw ClientStateError(myerror.str());
                }
                break;
 
-       case InitDone:
+       case CS_InitDone:
                switch(event)
                {
-               case SetDefinitionsSent:
-                       m_state = DefinitionsSent;
+               case CSE_SetDefinitionsSent:
+                       m_state = CS_DefinitionsSent;
                        break;
-
-               case Disconnect:
-                       m_state = Disconnecting;
+               case CSE_Disconnect:
+                       m_state = CS_Disconnecting;
                        break;
-
-               case SetDenied:
-                       m_state = Denied;
+               case CSE_SetDenied:
+                       m_state = CS_Denied;
                        break;
 
                /* Init GotInit2 SetMediaSent */
                default:
-                       assert("Invalid client state transition!" == 0);
+                       myerror << "InitDone: Invalid client state transition! " << event;
+                       throw ClientStateError(myerror.str());
                }
                break;
-
-       case DefinitionsSent:
+       case CS_DefinitionsSent:
                switch(event)
                {
-               case SetMediaSent:
-                       m_state = Active;
+               case CSE_SetClientReady:
+                       m_state = CS_Active;
                        break;
-
-               case Disconnect:
-                       m_state = Disconnecting;
+               case CSE_Disconnect:
+                       m_state = CS_Disconnecting;
                        break;
-
-               case SetDenied:
-                       m_state = Denied;
+               case CSE_SetDenied:
+                       m_state = CS_Denied;
                        break;
-
                /* Init GotInit2 SetDefinitionsSent */
                default:
-                       assert("Invalid client state transition!" == 0);
+                       myerror << "DefinitionsSent: Invalid client state transition! " << event;
+                       throw ClientStateError(myerror.str());
                }
                break;
-
-       case Active:
+       case CS_Active:
                switch(event)
                {
-               case SetDenied:
-                       m_state = Denied;
+               case CSE_SetDenied:
+                       m_state = CS_Denied;
                        break;
-
-               case Disconnect:
-                       m_state = Disconnecting;
+               case CSE_Disconnect:
+                       m_state = CS_Disconnecting;
+                       break;
+               case CSE_SudoSuccess:
+                       m_state = CS_SudoMode;
+                       if ((chosen_mech == AUTH_MECHANISM_SRP)
+                                       || (chosen_mech == AUTH_MECHANISM_LEGACY_PASSWORD))
+                               srp_verifier_delete((SRPVerifier *) auth_data);
+                       chosen_mech = AUTH_MECHANISM_NONE;
                        break;
-
                /* Init GotInit2 SetDefinitionsSent SetMediaSent SetDenied */
                default:
-                       assert("Invalid client state transition!" == 0);
+                       myerror << "Active: Invalid client state transition! " << event;
+                       throw ClientStateError(myerror.str());
                        break;
                }
                break;
-
-       case Disconnecting:
+       case CS_SudoMode:
+               switch(event)
+               {
+               case CSE_SetDenied:
+                       m_state = CS_Denied;
+                       break;
+               case CSE_Disconnect:
+                       m_state = CS_Disconnecting;
+                       break;
+               case CSE_SudoLeave:
+                       m_state = CS_Active;
+                       break;
+               default:
+                       myerror << "Active: Invalid client state transition! " << event;
+                       throw ClientStateError(myerror.str());
+                       break;
+               }
+               break;
+       case CS_Disconnecting:
                /* we are already disconnecting */
                break;
        }
 }
 
+u32 RemoteClient::uptime()
+{
+       return getTime(PRECISION_SECONDS) - m_connection_time;
+}
+
 ClientInterface::ClientInterface(con::Connection* con)
 :
        m_con(con),
@@ -529,28 +607,23 @@ ClientInterface::~ClientInterface()
                Delete clients
        */
        {
-               JMutexAutoLock clientslock(m_clients_mutex);
-
-               for(std::map<u16, RemoteClient*>::iterator
-                       i = m_clients.begin();
-                       i != m_clients.end(); ++i)
-               {
+               MutexAutoLock clientslock(m_clients_mutex);
 
+               for (UNORDERED_MAP<u16, RemoteClient*>::iterator i = m_clients.begin();
+                       i != m_clients.end(); ++i) {
                        // Delete client
                        delete i->second;
                }
        }
 }
 
-std::list<u16> ClientInterface::getClientIDs(ClientState min_state)
+std::vector<u16> ClientInterface::getClientIDs(ClientState min_state)
 {
-       std::list<u16> reply;
-       JMutexAutoLock clientslock(m_clients_mutex);
+       std::vector<u16> reply;
+       MutexAutoLock clientslock(m_clients_mutex);
 
-       for(std::map<u16, RemoteClient*>::iterator
-               i = m_clients.begin();
-               i != m_clients.end(); ++i)
-       {
+       for(UNORDERED_MAP<u16, RemoteClient*>::iterator i = m_clients.begin();
+               i != m_clients.end(); ++i) {
                if (i->second->getState() >= min_state)
                        reply.push_back(i->second->peer_id);
        }
@@ -558,12 +631,6 @@ std::list<u16> ClientInterface::getClientIDs(ClientState min_state)
        return reply;
 }
 
-std::vector<std::string> ClientInterface::getPlayerNames()
-{
-       return m_clients_names;
-}
-
-
 void ClientInterface::step(float dtime)
 {
        m_print_info_timer += dtime;
@@ -576,65 +643,61 @@ void ClientInterface::step(float dtime)
 
 void ClientInterface::UpdatePlayerList()
 {
-       if (m_env != NULL)
-               {
-               std::list<u16> clients = getClientIDs();
+       if (m_env != NULL) {
+               std::vector<u16> clients = getClientIDs();
                m_clients_names.clear();
 
 
-               if(clients.size() != 0)
+               if(!clients.empty())
                        infostream<<"Players:"<<std::endl;
-               for(std::list<u16>::iterator
-                       i = clients.begin();
-                       i != clients.end(); ++i)
-               {
-                       Player *player = m_env->getPlayer(*i);
-                       if(player==NULL)
+
+               for (std::vector<u16>::iterator i = clients.begin(); i != clients.end(); ++i) {
+                       RemotePlayer *player = m_env->getPlayer(*i);
+
+                       if (player == NULL)
                                continue;
-                       infostream<<"* "<<player->getName()<<"\t";
+
+                       infostream << "* " << player->getName() << "\t";
 
                        {
-                               JMutexAutoLock clientslock(m_clients_mutex);
+                               MutexAutoLock clientslock(m_clients_mutex);
                                RemoteClient* client = lockedGetClientNoEx(*i);
                                if(client != NULL)
                                        client->PrintInfo(infostream);
                        }
+
                        m_clients_names.push_back(player->getName());
                }
        }
 }
 
-void ClientInterface::send(u16 peer_id,u8 channelnum,
-               SharedBuffer<u8> data, bool reliable)
+void ClientInterface::send(u16 peer_id, u8 channelnum,
+               NetworkPacket* pkt, bool reliable)
 {
-       m_con->Send(peer_id, channelnum, data, reliable);
+       m_con->Send(peer_id, channelnum, pkt, reliable);
 }
 
 void ClientInterface::sendToAll(u16 channelnum,
-               SharedBuffer<u8> data, bool reliable)
+               NetworkPacket* pkt, bool reliable)
 {
-       JMutexAutoLock clientslock(m_clients_mutex);
-       for(std::map<u16, RemoteClient*>::iterator
-               i = m_clients.begin();
-               i != m_clients.end(); ++i)
-       {
+       MutexAutoLock clientslock(m_clients_mutex);
+       for(UNORDERED_MAP<u16, RemoteClient*>::iterator i = m_clients.begin();
+               i != m_clients.end(); ++i) {
                RemoteClient *client = i->second;
 
-               if (client->net_proto_version != 0)
-               {
-                       m_con->Send(client->peer_id, channelnum, data, reliable);
+               if (client->net_proto_version != 0) {
+                       m_con->Send(client->peer_id, channelnum, pkt, reliable);
                }
        }
 }
 
 RemoteClient* ClientInterface::getClientNoEx(u16 peer_id, ClientState state_min)
 {
-       JMutexAutoLock clientslock(m_clients_mutex);
-       std::map<u16, RemoteClient*>::iterator n;
-       n = m_clients.find(peer_id);
+       MutexAutoLock clientslock(m_clients_mutex);
+       UNORDERED_MAP<u16, RemoteClient*>::iterator n = m_clients.find(peer_id);
        // The client may not exist; clients are immediately removed if their
        // access is denied, and this event occurs later then.
-       if(n == m_clients.end())
+       if (n == m_clients.end())
                return NULL;
 
        if (n->second->getState() >= state_min)
@@ -645,11 +708,10 @@ RemoteClient* ClientInterface::getClientNoEx(u16 peer_id, ClientState state_min)
 
 RemoteClient* ClientInterface::lockedGetClientNoEx(u16 peer_id, ClientState state_min)
 {
-       std::map<u16, RemoteClient*>::iterator n;
-       n = m_clients.find(peer_id);
+       UNORDERED_MAP<u16, RemoteClient*>::iterator n = m_clients.find(peer_id);
        // The client may not exist; clients are immediately removed if their
        // access is denied, and this event occurs later then.
-       if(n == m_clients.end())
+       if (n == m_clients.end())
                return NULL;
 
        if (n->second->getState() >= state_min)
@@ -660,38 +722,35 @@ RemoteClient* ClientInterface::lockedGetClientNoEx(u16 peer_id, ClientState stat
 
 ClientState ClientInterface::getClientState(u16 peer_id)
 {
-       JMutexAutoLock clientslock(m_clients_mutex);
-       std::map<u16, RemoteClient*>::iterator n;
-       n = m_clients.find(peer_id);
+       MutexAutoLock clientslock(m_clients_mutex);
+       UNORDERED_MAP<u16, RemoteClient*>::iterator n = m_clients.find(peer_id);
        // The client may not exist; clients are immediately removed if their
        // access is denied, and this event occurs later then.
-       if(n == m_clients.end())
-               return Invalid;
+       if (n == m_clients.end())
+               return CS_Invalid;
 
        return n->second->getState();
 }
 
 void ClientInterface::setPlayerName(u16 peer_id,std::string name)
 {
-       JMutexAutoLock clientslock(m_clients_mutex);
-       std::map<u16, RemoteClient*>::iterator n;
-       n = m_clients.find(peer_id);
+       MutexAutoLock clientslock(m_clients_mutex);
+       UNORDERED_MAP<u16, RemoteClient*>::iterator n = m_clients.find(peer_id);
        // The client may not exist; clients are immediately removed if their
        // access is denied, and this event occurs later then.
-       if(n != m_clients.end())
+       if (n != m_clients.end())
                n->second->setName(name);
 }
 
 void ClientInterface::DeleteClient(u16 peer_id)
 {
-       JMutexAutoLock conlock(m_clients_mutex);
+       MutexAutoLock conlock(m_clients_mutex);
 
        // Error check
-       std::map<u16, RemoteClient*>::iterator n;
-       n = m_clients.find(peer_id);
+       UNORDERED_MAP<u16, RemoteClient*>::iterator n = m_clients.find(peer_id);
        // The client may not exist; clients are immediately removed if their
        // access is denied, and this event occurs later then.
-       if(n == m_clients.end())
+       if (n == m_clients.end())
                return;
 
        /*
@@ -719,13 +778,12 @@ void ClientInterface::DeleteClient(u16 peer_id)
 
 void ClientInterface::CreateClient(u16 peer_id)
 {
-       JMutexAutoLock conlock(m_clients_mutex);
+       MutexAutoLock conlock(m_clients_mutex);
 
        // Error check
-       std::map<u16, RemoteClient*>::iterator n;
-       n = m_clients.find(peer_id);
+       UNORDERED_MAP<u16, RemoteClient*>::iterator n = m_clients.find(peer_id);
        // The client shouldn't already exist
-       if(n != m_clients.end()) return;
+       if (n != m_clients.end()) return;
 
        // Create client
        RemoteClient *client = new RemoteClient();
@@ -736,11 +794,10 @@ void ClientInterface::CreateClient(u16 peer_id)
 void ClientInterface::event(u16 peer_id, ClientStateEvent event)
 {
        {
-               JMutexAutoLock clientlock(m_clients_mutex);
+               MutexAutoLock clientlock(m_clients_mutex);
 
                // Error check
-               std::map<u16, RemoteClient*>::iterator n;
-               n = m_clients.find(peer_id);
+               UNORDERED_MAP<u16, RemoteClient*>::iterator n = m_clients.find(peer_id);
 
                // No client to deliver event
                if (n == m_clients.end())
@@ -748,7 +805,9 @@ void ClientInterface::event(u16 peer_id, ClientStateEvent event)
                n->second->notifyEvent(event);
        }
 
-       if ((event == SetMediaSent) || (event == Disconnect) || (event == SetDenied))
+       if ((event == CSE_SetClientReady) ||
+               (event == CSE_Disconnect)     ||
+               (event == CSE_SetDenied))
        {
                UpdatePlayerList();
        }
@@ -756,15 +815,28 @@ void ClientInterface::event(u16 peer_id, ClientStateEvent event)
 
 u16 ClientInterface::getProtocolVersion(u16 peer_id)
 {
-       JMutexAutoLock conlock(m_clients_mutex);
+       MutexAutoLock conlock(m_clients_mutex);
 
        // Error check
-       std::map<u16, RemoteClient*>::iterator n;
-       n = m_clients.find(peer_id);
+       UNORDERED_MAP<u16, RemoteClient*>::iterator n = m_clients.find(peer_id);
 
-       // No client to deliver event
+       // No client to get version
        if (n == m_clients.end())
                return 0;
 
        return n->second->net_proto_version;
 }
+
+void ClientInterface::setClientVersion(u16 peer_id, u8 major, u8 minor, u8 patch, std::string full)
+{
+       MutexAutoLock conlock(m_clients_mutex);
+
+       // Error check
+       UNORDERED_MAP<u16, RemoteClient*>::iterator n = m_clients.find(peer_id);
+
+       // No client to set versions
+       if (n == m_clients.end())
+               return;
+
+       n->second->setVersionInfo(major,minor,patch,full);
+}