]> git.lizzy.rs Git - dragonfireclient.git/blobdiff - src/client.cpp
Make dungeon masters though and make oerkkis disappear when they get to you (because...
[dragonfireclient.git] / src / client.cpp
index ec2504bc823f45920f9d82335eb7181696e47e58..c40c9e5136db6eff9f450fca75c84cfbfca2d296 100644 (file)
@@ -25,6 +25,107 @@ with this program; if not, write to the Free Software Foundation, Inc.,
 #include "main.h"
 #include <sstream>
 #include "porting.h"
+#include "mapsector.h"
+#include "mapblock_mesh.h"
+#include "mapblock.h"
+#include "settings.h"
+#include "profiler.h"
+
+/*
+       QueuedMeshUpdate
+*/
+
+QueuedMeshUpdate::QueuedMeshUpdate():
+       p(-1337,-1337,-1337),
+       data(NULL),
+       ack_block_to_server(false)
+{
+}
+
+QueuedMeshUpdate::~QueuedMeshUpdate()
+{
+       if(data)
+               delete data;
+}
+
+/*
+       MeshUpdateQueue
+*/
+       
+MeshUpdateQueue::MeshUpdateQueue()
+{
+       m_mutex.Init();
+}
+
+MeshUpdateQueue::~MeshUpdateQueue()
+{
+       JMutexAutoLock lock(m_mutex);
+
+       core::list<QueuedMeshUpdate*>::Iterator i;
+       for(i=m_queue.begin(); i!=m_queue.end(); i++)
+       {
+               QueuedMeshUpdate *q = *i;
+               delete q;
+       }
+}
+
+/*
+       peer_id=0 adds with nobody to send to
+*/
+void MeshUpdateQueue::addBlock(v3s16 p, MeshMakeData *data, bool ack_block_to_server)
+{
+       DSTACK(__FUNCTION_NAME);
+
+       assert(data);
+
+       JMutexAutoLock lock(m_mutex);
+
+       /*
+               Find if block is already in queue.
+               If it is, update the data and quit.
+       */
+       core::list<QueuedMeshUpdate*>::Iterator i;
+       for(i=m_queue.begin(); i!=m_queue.end(); i++)
+       {
+               QueuedMeshUpdate *q = *i;
+               if(q->p == p)
+               {
+                       if(q->data)
+                               delete q->data;
+                       q->data = data;
+                       if(ack_block_to_server)
+                               q->ack_block_to_server = true;
+                       return;
+               }
+       }
+       
+       /*
+               Add the block
+       */
+       QueuedMeshUpdate *q = new QueuedMeshUpdate;
+       q->p = p;
+       q->data = data;
+       q->ack_block_to_server = ack_block_to_server;
+       m_queue.push_back(q);
+}
+
+// Returned pointer must be deleted
+// Returns NULL if queue is empty
+QueuedMeshUpdate * MeshUpdateQueue::pop()
+{
+       JMutexAutoLock lock(m_mutex);
+
+       core::list<QueuedMeshUpdate*>::Iterator i = m_queue.begin();
+       if(i == m_queue.end())
+               return NULL;
+       QueuedMeshUpdate *q = *i;
+       m_queue.erase(i);
+       return q;
+}
+
+/*
+       MeshUpdateThread
+*/
 
 void * MeshUpdateThread::Thread()
 {
@@ -36,13 +137,24 @@ void * MeshUpdateThread::Thread()
 
        while(getRun())
        {
+               /*// Wait for output queue to flush.
+               // Allow 2 in queue, this makes less frametime jitter.
+               // Umm actually, there is no much difference
+               if(m_queue_out.size() >= 2)
+               {
+                       sleep_ms(3);
+                       continue;
+               }*/
+
                QueuedMeshUpdate *q = m_queue_in.pop();
                if(q == NULL)
                {
-                       sleep_ms(50);
+                       sleep_ms(3);
                        continue;
                }
 
+               ScopeProfiler sp(g_profiler, "mesh make");
+
                scene::SMesh *mesh_new = NULL;
                mesh_new = makeMapBlockMesh(q->data);
 
@@ -68,6 +180,7 @@ void * MeshUpdateThread::Thread()
 Client::Client(
                IrrlichtDevice *device,
                const char *playername,
+               std::string password,
                MapDrawControl &control):
        m_mesh_update_thread(),
        m_env(
@@ -78,15 +191,15 @@ Client::Client(
        ),
        m_con(PROTOCOL_ID, 512, CONNECTION_TIMEOUT, this),
        m_device(device),
-       camera_position(0,0,0),
-       camera_direction(0,0,1),
        m_server_ser_ver(SER_FMT_VER_INVALID),
        m_inventory_updated(false),
        m_time_of_day(0),
-       m_map_seed(0)
+       m_map_seed(0),
+       m_password(password),
+       m_access_denied(false)
 {
        m_packetcounter_timer = 0.0;
-       m_delete_unused_sectors_timer = 0.0;
+       //m_delete_unused_sectors_timer = 0.0;
        m_connection_reinit_timer = 0.0;
        m_avg_rtt_timer = 0.0;
        m_playerpos_send_timer = 0.0;
@@ -190,7 +303,11 @@ void Client::step(float dtime)
                        m_packetcounter.clear();
                }
        }
+       
+       // Get connection status
+       bool connected = connectedAndInitialized();
 
+#if 0
        {
                /*
                        Delete unused sectors
@@ -212,24 +329,24 @@ void Client::step(float dtime)
                        core::list<v3s16> deleted_blocks;
 
                        float delete_unused_sectors_timeout = 
-                               g_settings.getFloat("client_delete_unused_sectors_timeout");
+                               g_settings->getFloat("client_delete_unused_sectors_timeout");
        
                        // Delete sector blocks
-                       /*u32 num = m_env.getMap().deleteUnusedSectors
+                       /*u32 num = m_env.getMap().unloadUnusedData
                                        (delete_unused_sectors_timeout,
                                        true, &deleted_blocks);*/
                        
                        // Delete whole sectors
-                       u32 num = m_env.getMap().deleteUnusedSectors
+                       m_env.getMap().unloadUnusedData
                                        (delete_unused_sectors_timeout,
-                                       false, &deleted_blocks);
+                                       &deleted_blocks);
 
-                       if(num > 0)
+                       if(deleted_blocks.size() > 0)
                        {
                                /*dstream<<DTIME<<"Client: Deleted blocks of "<<num
                                                <<" unused sectors"<<std::endl;*/
-                               dstream<<DTIME<<"Client: Deleted "<<num
-                                               <<" unused sectors"<<std::endl;
+                               /*dstream<<DTIME<<"Client: Deleted "<<num
+                                               <<" unused sectors"<<std::endl;*/
                                
                                /*
                                        Send info to server
@@ -279,8 +396,7 @@ void Client::step(float dtime)
                        }
                }
        }
-
-       bool connected = connectedAndInitialized();
+#endif
 
        if(connected == false)
        {
@@ -299,11 +415,24 @@ void Client::step(float dtime)
                        // [0] u16 TOSERVER_INIT
                        // [2] u8 SER_FMT_VER_HIGHEST
                        // [3] u8[20] player_name
-                       SharedBuffer<u8> data(2+1+PLAYERNAME_SIZE);
+                       // [23] u8[28] password (new in some version)
+                       // [51] u16 client network protocol version (new in some version)
+                       SharedBuffer<u8> data(2+1+PLAYERNAME_SIZE+PASSWORD_SIZE+2);
                        writeU16(&data[0], TOSERVER_INIT);
                        writeU8(&data[2], SER_FMT_VER_HIGHEST);
+
                        memset((char*)&data[3], 0, PLAYERNAME_SIZE);
                        snprintf((char*)&data[3], PLAYERNAME_SIZE, "%s", myplayer->getName());
+
+                       /*dstream<<"Client: sending initial password hash: \""<<m_password<<"\""
+                                       <<std::endl;*/
+
+                       memset((char*)&data[23], 0, PASSWORD_SIZE);
+                       snprintf((char*)&data[23], PASSWORD_SIZE, "%s", m_password.c_str());
+                       
+                       // This should be incremented in each version
+                       writeU16(&data[51], 3);
+
                        // Send as unreliable
                        Send(0, data, false);
                }
@@ -316,6 +445,67 @@ void Client::step(float dtime)
                Do stuff if connected
        */
        
+       /*
+               Run Map's timers and unload unused data
+       */
+       const float map_timer_and_unload_dtime = 5.25;
+       if(m_map_timer_and_unload_interval.step(dtime, map_timer_and_unload_dtime))
+       {
+               ScopeProfiler sp(g_profiler, "Client: map timer and unload");
+               core::list<v3s16> deleted_blocks;
+               m_env.getMap().timerUpdate(map_timer_and_unload_dtime,
+                               g_settings->getFloat("client_unload_unused_data_timeout"),
+                               &deleted_blocks);
+                               
+               /*if(deleted_blocks.size() > 0)
+                       dstream<<"Client: Unloaded "<<deleted_blocks.size()
+                                       <<" unused blocks"<<std::endl;*/
+                       
+               /*
+                       Send info to server
+                       NOTE: This loop is intentionally iterated the way it is.
+               */
+
+               core::list<v3s16>::Iterator i = deleted_blocks.begin();
+               core::list<v3s16> sendlist;
+               for(;;)
+               {
+                       if(sendlist.size() == 255 || i == deleted_blocks.end())
+                       {
+                               if(sendlist.size() == 0)
+                                       break;
+                               /*
+                                       [0] u16 command
+                                       [2] u8 count
+                                       [3] v3s16 pos_0
+                                       [3+6] v3s16 pos_1
+                                       ...
+                               */
+                               u32 replysize = 2+1+6*sendlist.size();
+                               SharedBuffer<u8> reply(replysize);
+                               writeU16(&reply[0], TOSERVER_DELETEDBLOCKS);
+                               reply[2] = sendlist.size();
+                               u32 k = 0;
+                               for(core::list<v3s16>::Iterator
+                                               j = sendlist.begin();
+                                               j != sendlist.end(); j++)
+                               {
+                                       writeV3S16(&reply[2+1+6*k], *j);
+                                       k++;
+                               }
+                               m_con.Send(PEER_ID_SERVER, 1, reply, true);
+
+                               if(i == deleted_blocks.end())
+                                       break;
+
+                               sendlist.clear();
+                       }
+
+                       sendlist.push_back(*i);
+                       i++;
+               }
+       }
+
        /*
                Handle environment
        */
@@ -331,25 +521,7 @@ void Client::step(float dtime)
                //TimeTaker envtimer("env step", m_device);
                // Step environment
                m_env.step(dtime);
-
-               // Step active blocks
-               for(core::map<v3s16, bool>::Iterator
-                               i = m_active_blocks.getIterator();
-                               i.atEnd() == false; i++)
-               {
-                       v3s16 p = i.getNode()->getKey();
-
-                       MapBlock *block = NULL;
-                       try
-                       {
-                               block = m_env.getMap().getBlockNoCreate(p);
-                               block->stepObjects(dtime, false, m_env.getDayNightRatio());
-                       }
-                       catch(InvalidPositionException &e)
-                       {
-                       }
-               }
-
+               
                /*
                        Get events
                */
@@ -546,8 +718,6 @@ void Client::ProcessData(u8 *data, u32 datasize, u16 sender_peer_id)
 
        //dstream<<"Client received command="<<(int)command<<std::endl;
 
-       // Execute fast commands straight away
-
        if(command == TOCLIENT_INIT)
        {
                if(datasize < 3)
@@ -599,7 +769,23 @@ void Client::ProcessData(u8 *data, u32 datasize, u16 sender_peer_id)
 
                return;
        }
-       
+
+       if(command == TOCLIENT_ACCESS_DENIED)
+       {
+               // The server didn't like our password. Note, this needs
+               // to be processed even if the serialisation format has
+               // not been agreed yet, the same as TOCLIENT_INIT.
+               m_access_denied = true;
+               m_access_denied_reason = L"Unknown";
+               if(datasize >= 4)
+               {
+                       std::string datastring((char*)&data[2], datasize-2);
+                       std::istringstream is(datastring, std::ios_base::binary);
+                       m_access_denied_reason = deSerializeWideString(is);
+               }
+               return;
+       }
+
        if(ser_version == SER_FMT_VER_INVALID)
        {
                dout_client<<DTIME<<"WARNING: Client: Server serialization"
@@ -667,78 +853,43 @@ void Client::ProcessData(u8 *data, u32 datasize, u16 sender_peer_id)
                MapSector *sector;
                MapBlock *block;
                
-               { //envlock
-                       //JMutexAutoLock envlock(m_env_mutex); //bulk comment-out
-                       
-                       v2s16 p2d(p.X, p.Z);
-                       sector = m_env.getMap().emergeSector(p2d);
-                       
-                       v2s16 sp = sector->getPos();
-                       if(sp != p2d)
-                       {
-                               dstream<<"ERROR: Got sector with getPos()="
-                                               <<"("<<sp.X<<","<<sp.Y<<"), tried to get"
-                                               <<"("<<p2d.X<<","<<p2d.Y<<")"<<std::endl;
-                       }
-
-                       assert(sp == p2d);
-                       //assert(sector->getPos() == p2d);
+               v2s16 p2d(p.X, p.Z);
+               sector = m_env.getMap().emergeSector(p2d);
+               
+               assert(sector->getPos() == p2d);
 
-                       //TimeTaker timer("MapBlock deSerialize");
-                       // 0ms
-                       
-                       try{
-                               block = sector->getBlockNoCreate(p.Y);
-                               /*
-                                       Update an existing block
-                               */
-                               //dstream<<"Updating"<<std::endl;
-                               block->deSerialize(istr, ser_version);
-                               //block->setChangedFlag();
-                       }
-                       catch(InvalidPositionException &e)
-                       {
-                               /*
-                                       Create a new block
-                               */
-                               //dstream<<"Creating new"<<std::endl;
-                               block = new MapBlock(&m_env.getMap(), p);
-                               block->deSerialize(istr, ser_version);
-                               sector->insertBlock(block);
-                               //block->setChangedFlag();
-
-                               //DEBUG
-                               /*NodeMod mod;
-                               mod.type = NODEMOD_CHANGECONTENT;
-                               mod.param = CONTENT_MESE;
-                               block->setTempMod(v3s16(8,10,8), mod);
-                               block->setTempMod(v3s16(8,9,8), mod);
-                               block->setTempMod(v3s16(8,8,8), mod);
-                               block->setTempMod(v3s16(8,7,8), mod);
-                               block->setTempMod(v3s16(8,6,8), mod);*/
-#if 0
-                               /*
-                                       Add some coulds
-                                       Well, this is a dumb way to do it, they should just
-                                       be drawn as separate objects. But the looks of them
-                                       can be tested this way.
-                               */
-                               if(p.Y == 3)
-                               {
-                                       NodeMod mod;
-                                       mod.type = NODEMOD_CHANGECONTENT;
-                                       mod.param = CONTENT_CLOUD;
-                                       v3s16 p2;
-                                       p2.Y = 8;
-                                       for(p2.X=3; p2.X<=13; p2.X++)
-                                       for(p2.Z=3; p2.Z<=13; p2.Z++)
-                                       {
-                                               block->setTempMod(p2, mod);
-                                       }
-                               }
-#endif
-                       }
-               } //envlock
+               //TimeTaker timer("MapBlock deSerialize");
+               // 0ms
+               
+               block = sector->getBlockNoCreateNoEx(p.Y);
+               if(block)
+               {
+                       /*
+                               Update an existing block
+                       */
+                       //dstream<<"Updating"<<std::endl;
+                       block->deSerialize(istr, ser_version);
+               }
+               else
+               {
+                       /*
+                               Create a new block
+                       */
+                       //dstream<<"Creating new"<<std::endl;
+                       block = new MapBlock(&m_env.getMap(), p);
+                       block->deSerialize(istr, ser_version);
+                       sector->insertBlock(block);
+
+                       //DEBUG
+                       /*NodeMod mod;
+                       mod.type = NODEMOD_CHANGECONTENT;
+                       mod.param = CONTENT_MESE;
+                       block->setTempMod(v3s16(8,10,8), mod);
+                       block->setTempMod(v3s16(8,9,8), mod);
+                       block->setTempMod(v3s16(8,8,8), mod);
+                       block->setTempMod(v3s16(8,7,8), mod);
+                       block->setTempMod(v3s16(8,6,8), mod);*/
+               }
 
 #if 0
                /*
@@ -767,7 +918,10 @@ void Client::ProcessData(u8 *data, u32 datasize, u16 sender_peer_id)
                */
 
                //m_env.getClientMap().updateMeshes(block->getPos(), getDayNightRatio());
-               
+               /*
+                       Add it to mesh update queue and set it to be acknowledged after update.
+               */
+               //std::cerr<<"Adding mesh update task for received block"<<std::endl;
                addUpdateMeshTaskWithEdge(p, true);
        }
        else if(command == TOCLIENT_PLAYERPOS)
@@ -944,6 +1098,8 @@ void Client::ProcessData(u8 *data, u32 datasize, u16 sender_peer_id)
        }
        else if(command == TOCLIENT_SECTORMETA)
        {
+               dstream<<"Client received DEPRECATED TOCLIENT_SECTORMETA"<<std::endl;
+#if 0
                /*
                        [0] u16 command
                        [2] u8 sector count
@@ -979,6 +1135,7 @@ void Client::ProcessData(u8 *data, u32 datasize, u16 sender_peer_id)
                                ((ClientMap&)m_env.getMap()).deSerializeSector(pos, is);
                        }
                } //envlock
+#endif
        }
        else if(command == TOCLIENT_INVENTORY)
        {
@@ -1016,16 +1173,11 @@ void Client::ProcessData(u8 *data, u32 datasize, u16 sender_peer_id)
        }
        //DEBUG
        else if(command == TOCLIENT_OBJECTDATA)
-       //else if(0)
        {
                // Strip command word and create a stringstream
                std::string datastring((char*)&data[2], datasize-2);
                std::istringstream is(datastring, std::ios_base::binary);
                
-               { //envlock
-               
-               //JMutexAutoLock envlock(m_env_mutex); //bulk comment-out
-
                u8 buf[12];
 
                /*
@@ -1075,136 +1227,39 @@ void Client::ProcessData(u8 *data, u32 datasize, u16 sender_peer_id)
 
                /*
                        Read block objects
+                       NOTE: Deprecated stuff
                */
 
                // Read active block count
-               is.read((char*)buf, 2);
-               u16 blockcount = readU16(buf);
-               
-               // Initialize delete queue with all active blocks
-               core::map<v3s16, bool> abs_to_delete;
-               for(core::map<v3s16, bool>::Iterator
-                               i = m_active_blocks.getIterator();
-                               i.atEnd() == false; i++)
-               {
-                       v3s16 p = i.getNode()->getKey();
-                       /*dstream<<"adding "
-                                       <<"("<<p.x<<","<<p.y<<","<<p.z<<") "
-                                       <<" to abs_to_delete"
-                                       <<std::endl;*/
-                       abs_to_delete.insert(p, true);
-               }
-
-               /*dstream<<"Initial delete queue size: "<<abs_to_delete.size()
-                               <<std::endl;*/
-               
-               for(u16 i=0; i<blockcount; i++)
-               {
-                       // Read blockpos
-                       is.read((char*)buf, 6);
-                       v3s16 p = readV3S16(buf);
-                       // Get block from somewhere
-                       MapBlock *block = NULL;
-                       try{
-                               block = m_env.getMap().getBlockNoCreate(p);
-                       }
-                       catch(InvalidPositionException &e)
-                       {
-                               //TODO: Create a dummy block?
-                       }
-                       if(block == NULL)
-                       {
-                               dstream<<"WARNING: "
-                                               <<"Could not get block at blockpos "
-                                               <<"("<<p.X<<","<<p.Y<<","<<p.Z<<") "
-                                               <<"in TOCLIENT_OBJECTDATA. Ignoring "
-                                               <<"following block object data."
-                                               <<std::endl;
-                               return;
-                       }
-
-                       /*dstream<<"Client updating objects for block "
-                                       <<"("<<p.X<<","<<p.Y<<","<<p.Z<<")"
-                                       <<std::endl;*/
-
-                       // Insert to active block list
-                       m_active_blocks.insert(p, true);
-
-                       // Remove from deletion queue
-                       if(abs_to_delete.find(p) != NULL)
-                               abs_to_delete.remove(p);
-
-                       /*
-                               Update objects of block
-                               
-                               NOTE: Be sure this is done in the main thread.
-                       */
-                       block->updateObjects(is, m_server_ser_ver,
-                                       m_device->getSceneManager(), m_env.getDayNightRatio());
-               }
-               
-               /*dstream<<"Final delete queue size: "<<abs_to_delete.size()
-                               <<std::endl;*/
-               
-               // Delete objects of blocks in delete queue
-               for(core::map<v3s16, bool>::Iterator
-                               i = abs_to_delete.getIterator();
-                               i.atEnd() == false; i++)
-               {
-                       v3s16 p = i.getNode()->getKey();
-                       try
-                       {
-                               MapBlock *block = m_env.getMap().getBlockNoCreate(p);
-                               
-                               // Clear objects
-                               block->clearObjects();
-                               // Remove from active blocks list
-                               m_active_blocks.remove(p);
-                       }
-                       catch(InvalidPositionException &e)
-                       {
-                               dstream<<"WARNAING: Client: "
-                                               <<"Couldn't clear objects of active->inactive"
-                                               <<" block "
-                                               <<"("<<p.X<<","<<p.Y<<","<<p.Z<<")"
-                                               <<" because block was not found"
-                                               <<std::endl;
-                               // Ignore
-                       }
+               u16 blockcount = readU16(is);
+               if(blockcount != 0){
+                       dstream<<"WARNING: TOCLIENT_OBJECTDATA: blockcount != 0 "
+                                       "not supported"<<std::endl;
+                       return;
                }
-
-               } //envlock
        }
        else if(command == TOCLIENT_TIME_OF_DAY)
        {
                if(datasize < 4)
                        return;
                
-               u16 time = readU16(&data[2]);
-               time = time % 24000;
-               m_time_of_day = time;
-               //dstream<<"Client: time="<<time<<std::endl;
+               u16 time_of_day = readU16(&data[2]);
+               time_of_day = time_of_day % 24000;
+               //dstream<<"Client: time_of_day="<<time_of_day<<std::endl;
                
                /*
-                       Day/night
-
                        time_of_day:
                        0 = midnight
                        12000 = midday
                */
                {
-                       u32 dr = time_to_daynight_ratio(m_time_of_day);
+                       m_env.setTimeOfDay(time_of_day);
+
+                       u32 dr = m_env.getDayNightRatio();
 
-                       dstream<<"Client: time_of_day="<<m_time_of_day
+                       dstream<<"Client: time_of_day="<<time_of_day
                                        <<", dr="<<dr
                                        <<std::endl;
-                       
-                       if(dr != m_env.getDayNightRatio())
-                       {
-                               dout_client<<DTIME<<"Client: changing day-night ratio"<<std::endl;
-                               m_env.setDayNightRatio(dr);
-                               m_env.expireMeshes(true);
-                       }
                }
 
        }
@@ -1237,7 +1292,7 @@ void Client::ProcessData(u8 *data, u32 datasize, u16 sender_peer_id)
        }
        else if(command == TOCLIENT_ACTIVE_OBJECT_REMOVE_ADD)
        {
-               //if(g_settings.getBool("enable_experimental"))
+               //if(g_settings->getBool("enable_experimental"))
                {
                        /*
                                u16 command
@@ -1249,7 +1304,7 @@ void Client::ProcessData(u8 *data, u32 datasize, u16 sender_peer_id)
                                for all added objects {
                                        u16 id
                                        u8 type
-                                       u16 initialization data length
+                                       u32 initialization data length
                                        string initialization data
                                }
                        */
@@ -1296,7 +1351,7 @@ void Client::ProcessData(u8 *data, u32 datasize, u16 sender_peer_id)
        }
        else if(command == TOCLIENT_ACTIVE_OBJECT_MESSAGES)
        {
-               //if(g_settings.getBool("enable_experimental"))
+               //if(g_settings->getBool("enable_experimental"))
                {
                        /*
                                u16 command
@@ -1381,6 +1436,63 @@ void Client::ProcessData(u8 *data, u32 datasize, u16 sender_peer_id)
                // get damage from falling on ground
                m_ignore_damage_timer = 3.0;
        }
+       else if(command == TOCLIENT_PLAYERITEM)
+       {
+               std::string datastring((char*)&data[2], datasize-2);
+               std::istringstream is(datastring, std::ios_base::binary);
+
+               u16 count = readU16(is);
+
+               for (u16 i = 0; i < count; ++i) {
+                       u16 peer_id = readU16(is);
+                       Player *player = m_env.getPlayer(peer_id);
+
+                       if (player == NULL)
+                       {
+                               dout_client<<DTIME<<"Client: ignoring player item "
+                                       << deSerializeString(is)
+                                       << " for non-existing peer id " << peer_id
+                                       << std::endl;
+                               continue;
+                       } else if (player->isLocal()) {
+                               dout_client<<DTIME<<"Client: ignoring player item "
+                                       << deSerializeString(is)
+                                       << " for local player" << std::endl;
+                               continue;
+                       } else {
+                               InventoryList *inv = player->inventory.getList("main");
+                               std::string itemstring(deSerializeString(is));
+                               if (itemstring.empty()) {
+                                       inv->deleteItem(0);
+                                       dout_client<<DTIME
+                                               <<"Client: empty player item for peer "
+                                               << peer_id << std::endl;
+                               } else {
+                                       std::istringstream iss(itemstring);
+                                       delete inv->changeItem(0, InventoryItem::deSerialize(iss));
+                                       dout_client<<DTIME<<"Client: player item for peer " << peer_id << ": ";
+                                       player->getWieldItem()->serialize(dout_client);
+                                       dout_client<<std::endl;
+                               }
+                       }
+               }
+       }
+       else if(command == TOCLIENT_DEATHSCREEN)
+       {
+               std::string datastring((char*)&data[2], datasize-2);
+               std::istringstream is(datastring, std::ios_base::binary);
+               
+               bool set_camera_point_target = readU8(is);
+               v3f camera_point_target = readV3F1000(is);
+               
+               ClientEvent event;
+               event.type = CE_DEATHSCREEN;
+               event.deathscreen.set_camera_point_target = set_camera_point_target;
+               event.deathscreen.camera_point_target_x = camera_point_target.X;
+               event.deathscreen.camera_point_target_y = camera_point_target.Y;
+               event.deathscreen.camera_point_target_z = camera_point_target.Z;
+               m_client_event_queue.push_back(event);
+       }
        else
        {
                dout_client<<DTIME<<"WARNING: Client: Ignoring unknown command "
@@ -1427,39 +1539,45 @@ void Client::groundAction(u8 action, v3s16 nodepos_undersurface,
        Send(0, data, true);
 }
 
-void Client::clickObject(u8 button, v3s16 blockpos, s16 id, u16 item)
+void Client::clickActiveObject(u8 button, u16 id, u16 item_i)
 {
        if(connectedAndInitialized() == false){
-               dout_client<<DTIME<<"Client::clickObject() "
+               dout_client<<DTIME<<"Client::clickActiveObject() "
                                "cancelled (not connected)"
                                <<std::endl;
                return;
        }
-       
-       /*
-               [0] u16 command=TOSERVER_CLICK_OBJECT
-               [2] u8 button (0=left, 1=right)
-               [3] v3s16 block
-               [9] s16 id
-               [11] u16 item
-       */
-       u8 datasize = 2 + 1 + 6 + 2 + 2;
-       SharedBuffer<u8> data(datasize);
-       writeU16(&data[0], TOSERVER_CLICK_OBJECT);
-       writeU8(&data[2], button);
-       writeV3S16(&data[3], blockpos);
-       writeS16(&data[9], id);
-       writeU16(&data[11], item);
-       Send(0, data, true);
-}
 
-void Client::clickActiveObject(u8 button, u16 id, u16 item)
-{
-       if(connectedAndInitialized() == false){
-               dout_client<<DTIME<<"Client::clickActiveObject() "
-                               "cancelled (not connected)"
-                               <<std::endl;
+       Player *player = m_env.getLocalPlayer();
+       if(player == NULL)
                return;
+
+       ClientActiveObject *obj = m_env.getActiveObject(id);
+       if(obj){
+               if(button == 0){
+                       ToolItem *titem = NULL;
+                       std::string toolname = "";
+
+                       InventoryList *mlist = player->inventory.getList("main");
+                       if(mlist != NULL)
+                       {
+                               InventoryItem *item = mlist->getItem(item_i);
+                               if(item && (std::string)item->getName() == "ToolItem")
+                               {
+                                       titem = (ToolItem*)item;
+                                       toolname = titem->getToolName();
+                               }
+                       }
+
+                       v3f playerpos = player->getPosition();
+                       v3f objpos = obj->getPosition();
+                       v3f dir = (objpos - playerpos).normalize();
+                       
+                       bool disable_send = obj->directReportPunch(toolname, dir);
+                       
+                       if(disable_send)
+                               return;
+               }
        }
        
        /*
@@ -1474,49 +1592,10 @@ void Client::clickActiveObject(u8 button, u16 id, u16 item)
        writeU16(&data[0], TOSERVER_CLICK_ACTIVEOBJECT);
        writeU8(&data[2], button);
        writeU16(&data[3], id);
-       writeU16(&data[5], item);
+       writeU16(&data[5], item_i);
        Send(0, data, true);
 }
 
-void Client::sendSignText(v3s16 blockpos, s16 id, std::string text)
-{
-       /*
-               u16 command
-               v3s16 blockpos
-               s16 id
-               u16 textlen
-               textdata
-       */
-       std::ostringstream os(std::ios_base::binary);
-       u8 buf[12];
-       
-       // Write command
-       writeU16(buf, TOSERVER_SIGNTEXT);
-       os.write((char*)buf, 2);
-       
-       // Write blockpos
-       writeV3S16(buf, blockpos);
-       os.write((char*)buf, 6);
-
-       // Write id
-       writeS16(buf, id);
-       os.write((char*)buf, 2);
-
-       u16 textlen = text.size();
-       // Write text length
-       writeS16(buf, textlen);
-       os.write((char*)buf, 2);
-
-       // Write text
-       os.write((char*)text.c_str(), textlen);
-       
-       // Make data buffer
-       std::string s = os.str();
-       SharedBuffer<u8> data((u8*)s.c_str(), s.size());
-       // Send as reliable
-       Send(0, data, true);
-}
-       
 void Client::sendSignNodeText(v3s16 p, std::string text)
 {
        /*
@@ -1597,6 +1676,43 @@ void Client::sendChatMessage(const std::wstring &message)
        Send(0, data, true);
 }
 
+void Client::sendChangePassword(const std::wstring oldpassword,
+               const std::wstring newpassword)
+{
+       Player *player = m_env.getLocalPlayer();
+       if(player == NULL)
+               return;
+
+       std::string playername = player->getName();
+       std::string oldpwd = translatePassword(playername, oldpassword);
+       std::string newpwd = translatePassword(playername, newpassword);
+
+       std::ostringstream os(std::ios_base::binary);
+       u8 buf[2+PASSWORD_SIZE*2];
+       /*
+               [0] u16 TOSERVER_PASSWORD
+               [2] u8[28] old password
+               [30] u8[28] new password
+       */
+
+       writeU16(buf, TOSERVER_PASSWORD);
+       for(u32 i=0;i<PASSWORD_SIZE-1;i++)
+       {
+               buf[2+i] = i<oldpwd.length()?oldpwd[i]:0;
+               buf[30+i] = i<newpwd.length()?newpwd[i]:0;
+       }
+       buf[2+PASSWORD_SIZE-1] = 0;
+       buf[30+PASSWORD_SIZE-1] = 0;
+       os.write((char*)buf, 2+PASSWORD_SIZE*2);
+
+       // Make data buffer
+       std::string s = os.str();
+       SharedBuffer<u8> data((u8*)s.c_str(), s.size());
+       // Send as reliable
+       Send(0, data, true);
+}
+
+
 void Client::sendDamage(u8 damage)
 {
        DSTACK(__FUNCTION_NAME);
@@ -1612,6 +1728,20 @@ void Client::sendDamage(u8 damage)
        Send(0, data, true);
 }
 
+void Client::sendRespawn()
+{
+       DSTACK(__FUNCTION_NAME);
+       std::ostringstream os(std::ios_base::binary);
+
+       writeU16(os, TOSERVER_RESPAWN);
+
+       // Make data buffer
+       std::string s = os.str();
+       SharedBuffer<u8> data((u8*)s.c_str(), s.size());
+       // Send as reliable
+       Send(0, data, true);
+}
+
 void Client::sendPlayerPos()
 {
        //JMutexAutoLock envlock(m_env_mutex); //bulk comment-out
@@ -1659,6 +1789,28 @@ void Client::sendPlayerPos()
        Send(0, data, false);
 }
 
+void Client::sendPlayerItem(u16 item)
+{
+       Player *myplayer = m_env.getLocalPlayer();
+       if(myplayer == NULL)
+               return;
+
+       u16 our_peer_id = m_con.GetPeerID();
+
+       // Set peer id if not set already
+       if(myplayer->peer_id == PEER_ID_INEXISTENT)
+               myplayer->peer_id = our_peer_id;
+       // Check that an existing peer_id is the same as the connection's
+       assert(myplayer->peer_id == our_peer_id);
+
+       SharedBuffer<u8> data(2+2);
+       writeU16(&data[0], TOSERVER_PLAYERITEM);
+       writeU16(&data[2], item);
+
+       // Send as reliable
+       Send(0, data, true);
+}
+
 void Client::removeNode(v3s16 p)
 {
        //JMutexAutoLock envlock(m_env_mutex); //bulk comment-out
@@ -1694,8 +1846,9 @@ void Client::addNode(v3s16 p, MapNode n)
 
        try
        {
-               TimeTaker timer3("Client::addNode(): addNodeAndUpdate");
-               m_env.getMap().addNodeAndUpdate(p, n, modified_blocks);
+               //TimeTaker timer3("Client::addNode(): addNodeAndUpdate");
+               std::string st = std::string("");
+               m_env.getMap().addNodeAndUpdate(p, n, modified_blocks, st);
        }
        catch(InvalidPositionException &e)
        {}
@@ -1712,11 +1865,14 @@ void Client::addNode(v3s16 p, MapNode n)
        }
 }
        
-void Client::updateCamera(v3f pos, v3f dir)
+void Client::updateCamera(v3f pos, v3f dir, f32 fov)
+{
+       m_env.getClientMap().updateCamera(pos, dir, fov);
+}
+
+void Client::renderPostFx()
 {
-       m_env.getClientMap().updateCamera(pos, dir);
-       camera_position = pos;
-       camera_direction = dir;
+       m_env.getClientMap().renderPostFx();
 }
 
 MapNode Client::getNode(v3s16 p)
@@ -1730,12 +1886,9 @@ NodeMetadata* Client::getNodeMetadata(v3s16 p)
        return m_env.getMap().getNodeMetadata(p);
 }
 
-v3f Client::getPlayerPosition()
+LocalPlayer* Client::getLocalPlayer()
 {
-       //JMutexAutoLock envlock(m_env_mutex); //bulk comment-out
-       LocalPlayer *player = m_env.getLocalPlayer();
-       assert(player != NULL);
-       return player->getPosition();
+       return m_env.getLocalPlayer();
 }
 
 void Client::setPlayerControl(PlayerControl &control)
@@ -1746,6 +1899,16 @@ void Client::setPlayerControl(PlayerControl &control)
        player->control = control;
 }
 
+void Client::selectPlayerItem(u16 item)
+{
+       LocalPlayer *player = m_env.getLocalPlayer();
+       assert(player != NULL);
+
+       player->wieldItem(item);
+
+       sendPlayerItem(item);
+}
+
 // Returns true if the inventory of the local player has been
 // updated from the server. If it is true, it is set to false.
 bool Client::getLocalInventoryUpdated()
@@ -1804,71 +1967,6 @@ void Client::inventoryAction(InventoryAction *a)
        sendInventoryAction(a);
 }
 
-MapBlockObject * Client::getSelectedObject(
-               f32 max_d,
-               v3f from_pos_f_on_map,
-               core::line3d<f32> shootline_on_map
-       )
-{
-       //JMutexAutoLock envlock(m_env_mutex); //bulk comment-out
-
-       core::array<DistanceSortedObject> objects;
-
-       for(core::map<v3s16, bool>::Iterator
-                       i = m_active_blocks.getIterator();
-                       i.atEnd() == false; i++)
-       {
-               v3s16 p = i.getNode()->getKey();
-
-               MapBlock *block = NULL;
-               try
-               {
-                       block = m_env.getMap().getBlockNoCreate(p);
-               }
-               catch(InvalidPositionException &e)
-               {
-                       continue;
-               }
-
-               // Calculate from_pos relative to block
-               v3s16 block_pos_i_on_map = block->getPosRelative();
-               v3f block_pos_f_on_map = intToFloat(block_pos_i_on_map, BS);
-               v3f from_pos_f_on_block = from_pos_f_on_map - block_pos_f_on_map;
-
-               block->getObjects(from_pos_f_on_block, max_d, objects);
-               //block->getPseudoObjects(from_pos_f_on_block, max_d, objects);
-       }
-
-       //dstream<<"Collected "<<objects.size()<<" nearby objects"<<std::endl;
-       
-       // Sort them.
-       // After this, the closest object is the first in the array.
-       objects.sort();
-
-       for(u32 i=0; i<objects.size(); i++)
-       {
-               MapBlockObject *obj = objects[i].obj;
-               MapBlock *block = obj->getBlock();
-
-               // Calculate shootline relative to block
-               v3s16 block_pos_i_on_map = block->getPosRelative();
-               v3f block_pos_f_on_map = intToFloat(block_pos_i_on_map, BS);
-               core::line3d<f32> shootline_on_block(
-                               shootline_on_map.start - block_pos_f_on_map,
-                               shootline_on_map.end - block_pos_f_on_map
-               );
-
-               if(obj->isSelected(shootline_on_block))
-               {
-                       //dstream<<"Returning selected object"<<std::endl;
-                       return obj;
-               }
-       }
-
-       //dstream<<"No object selected; returning NULL."<<std::endl;
-       return NULL;
-}
-
 ClientActiveObject * Client::getSelectedActiveObject(
                f32 max_d,
                v3f from_pos_f_on_map,
@@ -1922,12 +2020,6 @@ void Client::printDebugInfo(std::ostream &os)
                <<std::endl;*/
 }
        
-/*s32 Client::getDayNightIndex()
-{
-       assert(m_daynight_i >= 0 && m_daynight_i < DAYNIGHT_CACHE_COUNT);
-       return m_daynight_i;
-}*/
-
 u32 Client::getDayNightRatio()
 {
        //JMutexAutoLock envlock(m_env_mutex); //bulk comment-out
@@ -1941,6 +2033,40 @@ u16 Client::getHP()
        return player->hp;
 }
 
+void Client::setTempMod(v3s16 p, NodeMod mod)
+{
+       //JMutexAutoLock envlock(m_env_mutex); //bulk comment-out
+       assert(m_env.getMap().mapType() == MAPTYPE_CLIENT);
+
+       core::map<v3s16, MapBlock*> affected_blocks;
+       ((ClientMap&)m_env.getMap()).setTempMod(p, mod,
+                       &affected_blocks);
+
+       for(core::map<v3s16, MapBlock*>::Iterator
+                       i = affected_blocks.getIterator();
+                       i.atEnd() == false; i++)
+       {
+               i.getNode()->getValue()->updateMesh(m_env.getDayNightRatio());
+       }
+}
+
+void Client::clearTempMod(v3s16 p)
+{
+       //JMutexAutoLock envlock(m_env_mutex); //bulk comment-out
+       assert(m_env.getMap().mapType() == MAPTYPE_CLIENT);
+
+       core::map<v3s16, MapBlock*> affected_blocks;
+       ((ClientMap&)m_env.getMap()).clearTempMod(p,
+                       &affected_blocks);
+
+       for(core::map<v3s16, MapBlock*>::Iterator
+                       i = affected_blocks.getIterator();
+                       i.atEnd() == false; i++)
+       {
+               i.getNode()->getValue()->updateMesh(m_env.getDayNightRatio());
+       }
+}
+
 void Client::addUpdateMeshTask(v3s16 p, bool ack_to_server)
 {
        /*dstream<<"Client::addUpdateMeshTask(): "
@@ -1950,7 +2076,7 @@ void Client::addUpdateMeshTask(v3s16 p, bool ack_to_server)
        MapBlock *b = m_env.getMap().getBlockNoCreateNoEx(p);
        if(b == NULL)
                return;
-
+       
        /*
                Create a task to update the mesh of the block
        */
@@ -1959,7 +2085,8 @@ void Client::addUpdateMeshTask(v3s16 p, bool ack_to_server)
        
        {
                //TimeTaker timer("data fill");
-               // 0ms
+               // Release: ~0ms
+               // Debug: 1-6ms, avg=2ms
                data->fill(getDayNightRatio(), b);
        }
 
@@ -1985,6 +2112,10 @@ void Client::addUpdateMeshTask(v3s16 p, bool ack_to_server)
        }
 #endif
 
+       /*
+               Mark mesh as non-expired at this point so that it can already
+               be marked as expired again if the data changes
+       */
        b->setMeshExpired(false);
 }