]> git.lizzy.rs Git - minetest.git/blobdiff - src/client.cpp
Fix local map saving when joining a local server from the server tab
[minetest.git] / src / client.cpp
index b2709b6f0afae9c3215b728abee72173ec0d3902..cb6419fade8f2f0d7e5a4451a774a417543e3ab3 100644 (file)
@@ -17,13 +17,19 @@ with this program; if not, write to the Free Software Foundation, Inc.,
 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
 */
 
-#include "client.h"
 #include <iostream>
 #include <algorithm>
-#include "clientserver.h"
+#include <sstream>
+#include <IFileSystem.h>
 #include "jthread/jmutexautolock.h"
+#include "util/directiontables.h"
+#include "util/pointedthing.h"
+#include "util/serialize.h"
+#include "util/string.h"
+#include "strfnd.h"
+#include "client.h"
+#include "clientserver.h"
 #include "main.h"
-#include <sstream>
 #include "filesys.h"
 #include "porting.h"
 #include "mapsector.h"
@@ -37,19 +43,21 @@ with this program; if not, write to the Free Software Foundation, Inc.,
 #include "nodedef.h"
 #include "itemdef.h"
 #include "shader.h"
-#include <IFileSystem.h>
 #include "base64.h"
 #include "clientmap.h"
 #include "clientmedia.h"
 #include "sound.h"
-#include "util/string.h"
 #include "IMeshCache.h"
 #include "serialization.h"
-#include "util/serialize.h"
 #include "config.h"
-#include "util/directiontables.h"
-#include "util/pointedthing.h"
 #include "version.h"
+#include "drawscene.h"
+#include "subgame.h"
+#include "server.h"
+#include "database.h"
+#include "database-sqlite3.h"
+
+extern gui::IGUIEnvironment* guienv;
 
 /*
        QueuedMeshUpdate
@@ -71,7 +79,7 @@ QueuedMeshUpdate::~QueuedMeshUpdate()
 /*
        MeshUpdateQueue
 */
-       
+
 MeshUpdateQueue::MeshUpdateQueue()
 {
 }
@@ -122,7 +130,7 @@ void MeshUpdateQueue::addBlock(v3s16 p, MeshMakeData *data, bool ack_block_to_se
                        return;
                }
        }
-       
+
        /*
                Add the block
        */
@@ -165,20 +173,13 @@ void * MeshUpdateThread::Thread()
        log_register_thread("MeshUpdateThread");
 
        DSTACK(__FUNCTION_NAME);
-       
+
        BEGIN_DEBUG_EXCEPTION_HANDLER
 
+       porting::setThreadName("MeshUpdateThread");
+
        while(!StopRequested())
        {
-               /*// 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)
                {
@@ -188,7 +189,7 @@ void * MeshUpdateThread::Thread()
 
                ScopeProfiler sp(g_profiler, "Client: Mesh making");
 
-               MapBlockMesh *mesh_new = new MapBlockMesh(q->data);
+               MapBlockMesh *mesh_new = new MapBlockMesh(q->data, m_camera_offset);
                if(mesh_new->getMesh()->getMeshBufferCount() == 0)
                {
                        delete mesh_new;
@@ -200,10 +201,6 @@ void * MeshUpdateThread::Thread()
                r.mesh = mesh_new;
                r.ack_block_to_server = q->ack_block_to_server;
 
-               /*infostream<<"MeshUpdateThread: Processed "
-                               <<"("<<q->p.X<<","<<q->p.Y<<","<<q->p.Z<<")"
-                               <<std::endl;*/
-
                m_queue_out.push_back(r);
 
                delete q;
@@ -231,6 +228,11 @@ Client::Client(
                MtEventManager *event,
                bool ipv6
 ):
+       m_packetcounter_timer(0.0),
+       m_connection_reinit_timer(0.1),
+       m_avg_rtt_timer(0.0),
+       m_playerpos_send_timer(0.0),
+       m_ignore_damage_timer(0.0),
        m_tsrc(tsrc),
        m_shsrc(shsrc),
        m_itemdef(itemdef),
@@ -245,6 +247,7 @@ Client::Client(
                device->getSceneManager(),
                tsrc, this, device
        ),
+       m_particle_manager(&m_env),
        m_con(PROTOCOL_ID, 512, CONNECTION_TIMEOUT, ipv6, this),
        m_device(device),
        m_server_ser_ver(SER_FMT_VER_INVALID),
@@ -252,9 +255,11 @@ Client::Client(
        m_inventory_updated(false),
        m_inventory_from_server(NULL),
        m_inventory_from_server_age(0.0),
+       m_show_highlighted(false),
        m_animation_time(0),
        m_crack_level(-1),
        m_crack_pos(0,0,0),
+       m_highlighted_pos(0,0,0),
        m_map_seed(0),
        m_password(password),
        m_access_denied(false),
@@ -265,31 +270,30 @@ Client::Client(
        m_last_time_of_day_f(-1),
        m_time_of_day_update_timer(0),
        m_recommended_send_interval(0.1),
-       m_removed_sounds_check_timer(0)
+       m_removed_sounds_check_timer(0),
+       m_state(LC_Created)
 {
-       m_packetcounter_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;
-       m_ignore_damage_timer = 0.0;
-
        /*
                Add local player
        */
        {
-               Player *player = new LocalPlayer(this);
-
-               player->updateName(playername);
+               Player *player = new LocalPlayer(this, playername);
 
                m_env.addPlayer(player);
        }
+
+       m_cache_smooth_lighting = g_settings->getBool("smooth_lighting");
 }
 
 void Client::Stop()
 {
        //request all client managed threads to stop
        m_mesh_update_thread.Stop();
+       if (localdb != NULL) {
+               actionstream << "Local map saving ended" << std::endl;
+               localdb->endSave();
+               delete localserver;
+       }
 }
 
 bool Client::isShutdown()
@@ -302,10 +306,7 @@ bool Client::isShutdown()
 
 Client::~Client()
 {
-       {
-               //JMutexAutoLock conlock(m_con_mutex); //bulk comment-out
-               m_con.Disconnect();
-       }
+       m_con.Disconnect();
 
        m_mesh_update_thread.Stop();
        m_mesh_update_thread.Wait();
@@ -318,12 +319,10 @@ Client::~Client()
        delete m_inventory_from_server;
 
        // Delete detached inventories
-       {
-               for(std::map<std::string, Inventory*>::iterator
-                               i = m_detached_inventories.begin();
-                               i != m_detached_inventories.end(); i++){
-                       delete i->second;
-               }
+       for(std::map<std::string, Inventory*>::iterator
+                       i = m_detached_inventories.begin();
+                       i != m_detached_inventories.end(); i++){
+               delete i->second;
        }
 
        // cleanup 3d model meshes on client shutdown
@@ -336,60 +335,38 @@ Client::~Client()
        }
 }
 
-void Client::connect(Address address)
+void Client::connect(Address address,
+               const std::string &address_name,
+               bool is_local_server)
 {
        DSTACK(__FUNCTION_NAME);
-       //JMutexAutoLock lock(m_con_mutex); //bulk comment-out
-       m_con.SetTimeoutMs(0);
-       m_con.Connect(address);
-}
 
-bool Client::connectedAndInitialized()
-{
-       //JMutexAutoLock lock(m_con_mutex); //bulk comment-out
+       initLocalMapSaving(address, address_name, is_local_server);
 
-       if(m_con.Connected() == false)
-               return false;
-       
-       if(m_server_ser_ver == SER_FMT_VER_INVALID)
-               return false;
-       
-       return true;
+       m_con.SetTimeoutMs(0);
+       m_con.Connect(address);
 }
 
 void Client::step(float dtime)
 {
        DSTACK(__FUNCTION_NAME);
-       
+
        // Limit a bit
        if(dtime > 2.0)
                dtime = 2.0;
-       
+
        if(m_ignore_damage_timer > dtime)
                m_ignore_damage_timer -= dtime;
        else
                m_ignore_damage_timer = 0.0;
-       
+
        m_animation_time += dtime;
        if(m_animation_time > 60.0)
                m_animation_time -= 60.0;
 
        m_time_of_day_update_timer += dtime;
-       
-       //infostream<<"Client steps "<<dtime<<std::endl;
 
-       {
-               //TimeTaker timer("ReceiveAll()", m_device);
-               // 0ms
-               ReceiveAll();
-       }
-       
-       {
-               //TimeTaker timer("m_con_mutex + m_con.RunTimeouts()", m_device);
-               // 0ms
-               //JMutexAutoLock lock(m_con_mutex); //bulk comment-out
-               m_con.RunTimeouts(dtime);
-       }
+       ReceiveAll();
 
        /*
                Packet counter
@@ -400,15 +377,13 @@ void Client::step(float dtime)
                if(counter <= 0.0)
                {
                        counter = 20.0;
-                       
-                       infostream<<"Client packetcounter (20s):"<<std::endl;
+
+                       infostream << "Client packetcounter (" << m_packetcounter_timer
+                                       << "):"<<std::endl;
                        m_packetcounter.print(infostream);
                        m_packetcounter.clear();
                }
        }
-       
-       // Get connection status
-       bool connected = connectedAndInitialized();
 
 #if 0
        {
@@ -418,7 +393,7 @@ void Client::step(float dtime)
                        NOTE: This jams the game for a while because deleting sectors
                              clear caches
                */
-               
+
                float &counter = m_delete_unused_sectors_timer;
                counter -= dtime;
                if(counter <= 0.0)
@@ -433,12 +408,12 @@ void Client::step(float dtime)
 
                        float delete_unused_sectors_timeout =
                                g_settings->getFloat("client_delete_unused_sectors_timeout");
-       
+
                        // Delete sector blocks
                        /*u32 num = m_env.getMap().unloadUnusedData
                                        (delete_unused_sectors_timeout,
                                        true, &deleted_blocks);*/
-                       
+
                        // Delete whole sectors
                        m_env.getMap().unloadUnusedData
                                        (delete_unused_sectors_timeout,
@@ -450,14 +425,14 @@ void Client::step(float dtime)
                                                <<" unused sectors"<<std::endl;*/
                                /*infostream<<"Client: Deleted "<<num
                                                <<" unused sectors"<<std::endl;*/
-                               
+
                                /*
                                        Send info to server
                                */
 
                                // Env is locked so con can be locked.
                                //JMutexAutoLock lock(m_con_mutex); //bulk comment-out
-                               
+
                                core::list<v3s16>::Iterator i = deleted_blocks.begin();
                                core::list<v3s16> sendlist;
                                for(;;)
@@ -500,8 +475,13 @@ void Client::step(float dtime)
                }
        }
 #endif
-
-       if(connected == false)
+       // UGLY hack to fix 2 second startup delay caused by non existent
+       // server client startup synchronization in local server or singleplayer mode
+       static bool initial_step = true;
+       if (initial_step) {
+               initial_step = false;
+       }
+       else if(m_state == LC_Created)
        {
                float &counter = m_connection_reinit_timer;
                counter -= dtime;
@@ -510,10 +490,9 @@ void Client::step(float dtime)
                        counter = 2.0;
 
                        //JMutexAutoLock envlock(m_env_mutex); //bulk comment-out
-                       
+
                        Player *myplayer = m_env.getLocalPlayer();
                        assert(myplayer != NULL);
-       
                        // Send TOSERVER_INIT
                        // [0] u16 TOSERVER_INIT
                        // [2] u8 SER_FMT_VER_HIGHEST_READ
@@ -533,7 +512,7 @@ void Client::step(float dtime)
 
                        memset((char*)&data[23], 0, PASSWORD_SIZE);
                        snprintf((char*)&data[23], PASSWORD_SIZE, "%s", m_password.c_str());
-                       
+
                        writeU16(&data[51], CLIENT_PROTOCOL_VERSION_MIN);
                        writeU16(&data[53], CLIENT_PROTOCOL_VERSION_MAX);
 
@@ -548,7 +527,7 @@ void Client::step(float dtime)
        /*
                Do stuff if connected
        */
-       
+
        /*
                Run Map's timers and unload unused data
        */
@@ -560,11 +539,11 @@ void Client::step(float dtime)
                m_env.getMap().timerUpdate(map_timer_and_unload_dtime,
                                g_settings->getFloat("client_unload_unused_data_timeout"),
                                &deleted_blocks);
-                               
+
                /*if(deleted_blocks.size() > 0)
                        infostream<<"Client: Unloaded "<<deleted_blocks.size()
                                        <<" unused blocks"<<std::endl;*/
-                       
+
                /*
                        Send info to server
                        NOTE: This loop is intentionally iterated the way it is.
@@ -576,7 +555,7 @@ void Client::step(float dtime)
                {
                        if(sendlist.size() == 255 || i == deleted_blocks.end())
                        {
-                               if(sendlist.size() == 0)
+                               if(sendlist.empty())
                                        break;
                                /*
                                        [0] u16 command
@@ -614,18 +593,14 @@ void Client::step(float dtime)
                Handle environment
        */
        {
-               // 0ms
-               //JMutexAutoLock lock(m_env_mutex); //bulk comment-out
-
                // Control local player (0ms)
                LocalPlayer *player = m_env.getLocalPlayer();
                assert(player != NULL);
                player->applyControl(dtime);
 
-               //TimeTaker envtimer("env step", m_device);
                // Step environment
                m_env.step(dtime);
-               
+
                /*
                        Get events
                */
@@ -641,7 +616,7 @@ void Client::step(float dtime)
                                if(m_ignore_damage_timer <= 0)
                                {
                                        u8 damage = event.player_damage.amount;
-                                       
+
                                        if(event.player_damage.send_to_server)
                                                sendDamage(damage);
 
@@ -669,9 +644,8 @@ void Client::step(float dtime)
                if(counter >= 10)
                {
                        counter = 0.0;
-                       //JMutexAutoLock lock(m_con_mutex); //bulk comment-out
                        // connectedAndInitialized() is true, peer exists.
-                       float avg_rtt = m_con.GetPeerAvgRTT(PEER_ID_SERVER);
+                       float avg_rtt = getRTT();
                        infostream<<"Client: avg_rtt="<<avg_rtt<<std::endl;
                }
        }
@@ -682,7 +656,7 @@ void Client::step(float dtime)
        {
                float &counter = m_playerpos_send_timer;
                counter += dtime;
-               if(counter >= m_recommended_send_interval)
+               if((m_state == LC_Ready) && (counter >= m_recommended_send_interval))
                {
                        counter = 0.0;
                        sendPlayerPos();
@@ -693,15 +667,6 @@ void Client::step(float dtime)
                Replace updated meshes
        */
        {
-               //JMutexAutoLock lock(m_env_mutex); //bulk comment-out
-
-               //TimeTaker timer("** Processing mesh update result queue");
-               // 0ms
-               
-               /*infostream<<"Mesh update result queue size is "
-                               <<m_mesh_update_thread.m_queue_out.size()
-                               <<std::endl;*/
-               
                int num_processed_meshes = 0;
                while(!m_mesh_update_thread.m_queue_out.empty())
                {
@@ -710,8 +675,6 @@ void Client::step(float dtime)
                        MapBlock *block = m_env.getMap().getBlockNoCreateNoEx(r.p);
                        if(block)
                        {
-                               //JMutexAutoLock lock(block->mesh_mutex);
-
                                // Delete the old mesh
                                if(block->mesh != NULL)
                                {
@@ -727,8 +690,6 @@ void Client::step(float dtime)
                        }
                        if(r.ack_block_to_server)
                        {
-                               /*infostream<<"Client: ACK block ("<<r.p.X<<","<<r.p.Y
-                                               <<","<<r.p.Z<<")"<<std::endl;*/
                                /*
                                        Acknowledge block
                                */
@@ -758,6 +719,7 @@ void Client::step(float dtime)
        if (m_media_downloader && m_media_downloader->isStarted()) {
                m_media_downloader->step(this);
                if (m_media_downloader->isDone()) {
+                       received_media();
                        delete m_media_downloader;
                        m_media_downloader = NULL;
                }
@@ -804,7 +766,7 @@ void Client::step(float dtime)
                        m_sound->updateSoundPosition(client_id, pos);
                }
        }
-       
+
        /*
                Handle removed remotely initiated sounds
        */
@@ -829,11 +791,13 @@ void Client::step(float dtime)
                        }
                }
                // Sync to server
-               if(removed_server_ids.size() != 0)
+               if(!removed_server_ids.empty())
                {
                        std::ostringstream os(std::ios_base::binary);
                        writeU16(os, TOSERVER_REMOVED_SOUNDS);
-                       writeU16(os, removed_server_ids.size());
+                       size_t server_ids = removed_server_ids.size();
+                       assert(server_ids <= 0xFFFF);
+                       writeU16(os, (u16) (server_ids & 0xFFFF));
                        for(std::set<s32>::iterator i = removed_server_ids.begin();
                                        i != removed_server_ids.end(); i++)
                                writeS32(os, *i);
@@ -849,7 +813,7 @@ bool Client::loadMedia(const std::string &data, const std::string &filename)
 {
        // Silly irrlicht's const-incorrectness
        Buffer<char> data_rw(data.c_str(), data.size());
-       
+
        std::string name;
 
        const char *image_ext[] = {
@@ -946,7 +910,9 @@ void Client::request_media(const std::list<std::string> &file_requests)
 {
        std::ostringstream os(std::ios_base::binary);
        writeU16(os, TOSERVER_REQUEST_MEDIA);
-       writeU16(os, file_requests.size());
+       size_t file_requests_size = file_requests.size();
+       assert(file_requests_size <= 0xFFFF);
+       writeU16(os, (u16) (file_requests_size & 0xFFFF));
 
        for(std::list<std::string>::const_iterator i = file_requests.begin();
                        i != file_requests.end(); ++i) {
@@ -975,6 +941,41 @@ void Client::received_media()
                        <<std::endl;
 }
 
+void Client::initLocalMapSaving(const Address &address,
+               const std::string &hostname,
+               bool is_local_server)
+{
+       localdb = NULL;
+
+       if (!g_settings->getBool("enable_local_map_saving") || is_local_server)
+               return;
+
+       const std::string world_path = porting::path_user
+               + DIR_DELIM + "worlds"
+               + DIR_DELIM + "server_"
+               + hostname + "_" + to_string(address.getPort());
+
+       SubgameSpec gamespec;
+
+       if (!getWorldExists(world_path)) {
+               gamespec = findSubgame(g_settings->get("default_game"));
+               if (!gamespec.isValid())
+                       gamespec = findSubgame("minimal");
+       } else {
+               gamespec = findWorldSubgame(world_path);
+       }
+
+       if (!gamespec.isValid()) {
+               errorstream << "Couldn't find subgame for local map saving." << std::endl;
+               return;
+       }
+
+       localserver = new Server(world_path, gamespec, false, false);
+       localdb = new Database_SQLite3(&(ServerMap&)localserver->getMap(), world_path);
+       localdb->beginSave();
+       actionstream << "Local map saving started, map will be saved at '" << world_path << "'" << std::endl;
+}
+
 void Client::ReceiveAll()
 {
        DSTACK(__FUNCTION_NAME);
@@ -985,7 +986,7 @@ void Client::ReceiveAll()
                // process
                if(porting::getTimeMs() > start_ms + 100)
                        break;
-               
+
                try{
                        Receive();
                        g_profiler->graphAdd("client_received_packets", 1);
@@ -1008,13 +1009,7 @@ void Client::Receive()
        DSTACK(__FUNCTION_NAME);
        SharedBuffer<u8> data;
        u16 sender_peer_id;
-       u32 datasize;
-       {
-               //TimeTaker t1("con mutex and receive", m_device);
-               //JMutexAutoLock lock(m_con_mutex); //bulk comment-out
-               datasize = m_con.Receive(sender_peer_id, data);
-       }
-       //TimeTaker t1("ProcessData", m_device);
+       u32 datasize = m_con.Receive(sender_peer_id, data);
        ProcessData(*data, datasize, sender_peer_id);
 }
 
@@ -1036,7 +1031,7 @@ void Client::ProcessData(u8 *data, u32 datasize, u16 sender_peer_id)
 
        //infostream<<"Client: received command="<<command<<std::endl;
        m_packetcounter.add((u16)command);
-       
+
        /*
                If this check is removed, be sure to change the queue
                system to know the ids
@@ -1051,8 +1046,6 @@ void Client::ProcessData(u8 *data, u32 datasize, u16 sender_peer_id)
 
        u8 ser_version = m_server_ser_ver;
 
-       //infostream<<"Client received command="<<(int)command<<std::endl;
-
        if(command == TOCLIENT_INIT)
        {
                if(datasize < 3)
@@ -1069,7 +1062,7 @@ void Client::ProcessData(u8 *data, u32 datasize, u16 sender_peer_id)
                                        <<"unsupported ser_fmt_ver"<<std::endl;
                        return;
                }
-               
+
                m_server_ser_ver = deployed;
 
                // Get player position
@@ -1078,15 +1071,12 @@ void Client::ProcessData(u8 *data, u32 datasize, u16 sender_peer_id)
                        playerpos_s16 = readV3S16(&data[2+1]);
                v3f playerpos_f = intToFloat(playerpos_s16, BS) - v3f(0, BS/2, 0);
 
-               { //envlock
-                       //JMutexAutoLock envlock(m_env_mutex); //bulk comment-out
-                       
-                       // Set player position
-                       Player *player = m_env.getLocalPlayer();
-                       assert(player != NULL);
-                       player->setPosition(playerpos_f);
-               }
-               
+
+               // Set player position
+               Player *player = m_env.getLocalPlayer();
+               assert(player != NULL);
+               player->setPosition(playerpos_f);
+
                if(datasize >= 2+1+6+8)
                {
                        // Get map seed
@@ -1101,7 +1091,7 @@ void Client::ProcessData(u8 *data, u32 datasize, u16 sender_peer_id)
                        infostream<<"Client: received recommended send interval "
                                        <<m_recommended_send_interval<<std::endl;
                }
-               
+
                // Reply to server
                u32 replysize = 2;
                SharedBuffer<u8> reply(replysize);
@@ -1109,6 +1099,8 @@ void Client::ProcessData(u8 *data, u32 datasize, u16 sender_peer_id)
                // Send as reliable
                m_con.Send(PEER_ID_SERVER, 1, reply, true);
 
+               m_state = LC_Init;
+
                return;
        }
 
@@ -1135,10 +1127,14 @@ void Client::ProcessData(u8 *data, u32 datasize, u16 sender_peer_id)
                                " Skipping incoming command="<<command<<std::endl;
                return;
        }
-       
-       // Just here to avoid putting the two if's together when
-       // making some copypasta
-       {}
+
+       /*
+         Handle runtime commands
+       */
+       // there's no sane reason why we shouldn't have a player and
+       // almost everyone needs a player reference
+       Player *player = m_env.getLocalPlayer();
+       assert(player != NULL);
 
        if(command == TOCLIENT_REMOVENODE)
        {
@@ -1148,9 +1144,6 @@ void Client::ProcessData(u8 *data, u32 datasize, u16 sender_peer_id)
                p.X = readS16(&data[2]);
                p.Y = readS16(&data[4]);
                p.Z = readS16(&data[6]);
-               
-               //TimeTaker t1("TOCLIENT_REMOVENODE");
-               
                removeNode(p);
        }
        else if(command == TOCLIENT_ADDNODE)
@@ -1162,18 +1155,16 @@ void Client::ProcessData(u8 *data, u32 datasize, u16 sender_peer_id)
                p.X = readS16(&data[2]);
                p.Y = readS16(&data[4]);
                p.Z = readS16(&data[6]);
-               
-               //TimeTaker t1("TOCLIENT_ADDNODE");
 
                MapNode n;
                n.deSerialize(&data[8], ser_version);
-               
+
                bool remove_metadata = true;
                u32 index = 8 + MapNode::serializedLength(ser_version);
                if ((datasize >= index+1) && data[index]){
                        remove_metadata = false;
                }
-               
+
                addNode(p, n, remove_metadata);
        }
        else if(command == TOCLIENT_BLOCKDATA)
@@ -1181,38 +1172,29 @@ void Client::ProcessData(u8 *data, u32 datasize, u16 sender_peer_id)
                // Ignore too small packet
                if(datasize < 8)
                        return;
-                       
+
                v3s16 p;
                p.X = readS16(&data[2]);
                p.Y = readS16(&data[4]);
                p.Z = readS16(&data[6]);
-               
-               /*infostream<<"Client: Thread: BLOCKDATA for ("
-                               <<p.X<<","<<p.Y<<","<<p.Z<<")"<<std::endl;*/
-               /*infostream<<"Client: Thread: BLOCKDATA for ("
-                               <<p.X<<","<<p.Y<<","<<p.Z<<")"<<std::endl;*/
-               
+
                std::string datastring((char*)&data[8], datasize-8);
                std::istringstream istr(datastring, std::ios_base::binary);
-               
+
                MapSector *sector;
                MapBlock *block;
-               
+
                v2s16 p2d(p.X, p.Z);
                sector = m_env.getMap().emergeSector(p2d);
-               
+
                assert(sector->getPos() == p2d);
 
-               //TimeTaker timer("MapBlock deSerialize");
-               // 0ms
-               
                block = sector->getBlockNoCreateNoEx(p.Y);
                if(block)
                {
                        /*
                                Update an existing block
                        */
-                       //infostream<<"Updating"<<std::endl;
                        block->deSerialize(istr, ser_version, false);
                        block->deSerializeNetworkSpecific(istr);
                }
@@ -1221,37 +1203,19 @@ void Client::ProcessData(u8 *data, u32 datasize, u16 sender_peer_id)
                        /*
                                Create a new block
                        */
-                       //infostream<<"Creating new"<<std::endl;
                        block = new MapBlock(&m_env.getMap(), p, this);
                        block->deSerialize(istr, ser_version, false);
                        block->deSerializeNetworkSpecific(istr);
                        sector->insertBlock(block);
                }
 
-#if 0
-               /*
-                       Acknowledge block
-               */
-               /*
-                       [0] u16 command
-                       [2] u8 count
-                       [3] v3s16 pos_0
-                       [3+6] v3s16 pos_1
-                       ...
-               */
-               u32 replysize = 2+1+6;
-               SharedBuffer<u8> reply(replysize);
-               writeU16(&reply[0], TOSERVER_GOTBLOCKS);
-               reply[2] = 1;
-               writeV3S16(&reply[3], p);
-               // Send as reliable
-               m_con.Send(PEER_ID_SERVER, 1, reply, true);
-#endif
+               if (localdb != NULL) {
+                       ((ServerMap&) localserver->getMap()).saveBlock(block, localdb);
+               }
 
                /*
                        Add it to mesh update queue and set it to be acknowledged after update.
                */
-               //infostream<<"Adding mesh update task for received block"<<std::endl;
                addUpdateMeshTaskWithEdge(p, true);
        }
        else if(command == TOCLIENT_INVENTORY)
@@ -1259,74 +1223,60 @@ void Client::ProcessData(u8 *data, u32 datasize, u16 sender_peer_id)
                if(datasize < 3)
                        return;
 
-               //TimeTaker t1("Parsing TOCLIENT_INVENTORY", m_device);
-
-               { //envlock
-                       //TimeTaker t2("mutex locking", m_device);
-                       //JMutexAutoLock envlock(m_env_mutex); //bulk comment-out
-                       //t2.stop();
-                       
-                       //TimeTaker t3("istringstream init", m_device);
-                       std::string datastring((char*)&data[2], datasize-2);
-                       std::istringstream is(datastring, std::ios_base::binary);
-                       //t3.stop();
-                       
-                       //TimeTaker t4("player get", m_device);
-                       Player *player = m_env.getLocalPlayer();
-                       assert(player != NULL);
-                       //t4.stop();
+               std::string datastring((char*)&data[2], datasize-2);
+               std::istringstream is(datastring, std::ios_base::binary);
 
-                       //TimeTaker t1("inventory.deSerialize()", m_device);
-                       player->inventory.deSerialize(is);
-                       //t1.stop();
+               player->inventory.deSerialize(is);
 
-                       m_inventory_updated = true;
+               m_inventory_updated = true;
 
-                       delete m_inventory_from_server;
-                       m_inventory_from_server = new Inventory(player->inventory);
-                       m_inventory_from_server_age = 0.0;
+               delete m_inventory_from_server;
+               m_inventory_from_server = new Inventory(player->inventory);
+               m_inventory_from_server_age = 0.0;
 
-                       //infostream<<"Client got player inventory:"<<std::endl;
-                       //player->inventory.print(infostream);
-               }
        }
        else if(command == TOCLIENT_TIME_OF_DAY)
        {
                if(datasize < 4)
                        return;
-               
-               u16 time_of_day = readU16(&data[2]);
-               time_of_day = time_of_day % 24000;
-               //infostream<<"Client: time_of_day="<<time_of_day<<std::endl;
+
+               u16 time_of_day  = readU16(&data[2]);
+               time_of_day      = time_of_day % 24000;
                float time_speed = 0;
-               if(datasize >= 2 + 2 + 4){
+
+               if(datasize >= 2 + 2 + 4)
+               {
                        time_speed = readF1000(&data[4]);
-               } else {
+               }
+               else {
                        // Old message; try to approximate speed of time by ourselves
                        float time_of_day_f = (float)time_of_day / 24000.0;
                        float tod_diff_f = 0;
+
                        if(time_of_day_f < 0.2 && m_last_time_of_day_f > 0.8)
                                tod_diff_f = time_of_day_f - m_last_time_of_day_f + 1.0;
                        else
                                tod_diff_f = time_of_day_f - m_last_time_of_day_f;
-                       m_last_time_of_day_f = time_of_day_f;
-                       float time_diff = m_time_of_day_update_timer;
+
+                       m_last_time_of_day_f         = time_of_day_f;
+                       float time_diff            = m_time_of_day_update_timer;
                        m_time_of_day_update_timer = 0;
+
                        if(m_time_of_day_set){
-                               time_speed = 3600.0*24.0 * tod_diff_f / time_diff;
+                               time_speed = (3600.0*24.0) * tod_diff_f / time_diff;
                                infostream<<"Client: Measured time_of_day speed (old format): "
                                                <<time_speed<<" tod_diff_f="<<tod_diff_f
                                                <<" time_diff="<<time_diff<<std::endl;
                        }
                }
-               
+
                // Update environment
                m_env.setTimeOfDay(time_of_day);
                m_env.setTimeOfDaySpeed(time_speed);
                m_time_of_day_set = true;
 
                u32 dr = m_env.getDayNightRatio();
-               verbosestream<<"Client: time_of_day="<<time_of_day
+               infostream<<"Client: time_of_day="<<time_of_day
                                <<" time_speed="<<time_speed
                                <<" dr="<<dr<<std::endl;
        }
@@ -1340,153 +1290,128 @@ void Client::ProcessData(u8 *data, u32 datasize, u16 sender_peer_id)
                u8 buf[6];
                std::string datastring((char*)&data[2], datasize-2);
                std::istringstream is(datastring, std::ios_base::binary);
-               
+
                // Read stuff
-               is.read((char*)buf, 2);
+               is.read((char*) buf, 2);
                u16 len = readU16(buf);
-               
+
                std::wstring message;
-               for(u16 i=0; i<len; i++)
+               for(unsigned int i=0; i<len; i++)
                {
                        is.read((char*)buf, 2);
                        message += (wchar_t)readU16(buf);
                }
 
-               /*infostream<<"Client received chat message: "
-                               <<wide_to_narrow(message)<<std::endl;*/
-               
                m_chat_queue.push_back(message);
        }
        else if(command == TOCLIENT_ACTIVE_OBJECT_REMOVE_ADD)
        {
-               //if(g_settings->getBool("enable_experimental"))
-               {
-                       /*
-                               u16 command
-                               u16 count of removed objects
-                               for all removed objects {
-                                       u16 id
-                               }
-                               u16 count of added objects
-                               for all added objects {
-                                       u16 id
-                                       u8 type
-                                       u32 initialization data length
-                                       string initialization data
-                               }
-                       */
+               /*
+                       u16 command
+                       u16 count of removed objects
+                       for all removed objects {
+                               u16 id
+                       }
+                       u16 count of added objects
+                       for all added objects {
+                               u16 id
+                               u8 type
+                               u32 initialization data length
+                               string initialization data
+                       }
+               */
 
-                       char buf[6];
-                       // Get all data except the command number
-                       std::string datastring((char*)&data[2], datasize-2);
-                       // Throw them in an istringstream
-                       std::istringstream is(datastring, std::ios_base::binary);
+               char buf[6];
+               // Get all data except the command number
+               std::string datastring((char*)&data[2], datasize-2);
+               // Throw them in an istringstream
+               std::istringstream is(datastring, std::ios_base::binary);
 
-                       // Read stuff
-                       
-                       // Read removed objects
+               // Read removed objects
+               is.read(buf, 2);
+               u16 removed_count = readU16((u8*)buf);
+               for(unsigned int i=0; i<removed_count; i++)
+               {
                        is.read(buf, 2);
-                       u16 removed_count = readU16((u8*)buf);
-                       for(u16 i=0; i<removed_count; i++)
-                       {
-                               is.read(buf, 2);
-                               u16 id = readU16((u8*)buf);
-                               // Remove it
-                               {
-                                       //JMutexAutoLock envlock(m_env_mutex); //bulk comment-out
-                                       m_env.removeActiveObject(id);
-                               }
-                       }
-                       
-                       // Read added objects
+                       u16 id = readU16((u8*)buf);
+                       m_env.removeActiveObject(id);
+               }
+
+               // Read added objects
+               is.read(buf, 2);
+               u16 added_count = readU16((u8*)buf);
+               for(unsigned int i=0; i<added_count; i++)
+               {
                        is.read(buf, 2);
-                       u16 added_count = readU16((u8*)buf);
-                       for(u16 i=0; i<added_count; i++)
-                       {
-                               is.read(buf, 2);
-                               u16 id = readU16((u8*)buf);
-                               is.read(buf, 1);
-                               u8 type = readU8((u8*)buf);
-                               std::string data = deSerializeLongString(is);
-                               // Add it
-                               {
-                                       //JMutexAutoLock envlock(m_env_mutex); //bulk comment-out
-                                       m_env.addActiveObject(id, type, data);
-                               }
-                       }
+                       u16 id = readU16((u8*)buf);
+                       is.read(buf, 1);
+                       u8 type = readU8((u8*)buf);
+                       std::string data = deSerializeLongString(is);
+                       // Add it
+                       m_env.addActiveObject(id, type, data);
                }
        }
        else if(command == TOCLIENT_ACTIVE_OBJECT_MESSAGES)
        {
-               //if(g_settings->getBool("enable_experimental"))
+               /*
+                       u16 command
+                       for all objects
+                       {
+                               u16 id
+                               u16 message length
+                               string message
+                       }
+               */
+               char buf[6];
+               // Get all data except the command number
+               std::string datastring((char*)&data[2], datasize-2);
+               // Throw them in an istringstream
+               std::istringstream is(datastring, std::ios_base::binary);
+
+               while(is.eof() == false)
                {
-                       /*
-                               u16 command
-                               for all objects
-                               {
-                                       u16 id
-                                       u16 message length
-                                       string message
-                               }
-                       */
-                       char buf[6];
-                       // Get all data except the command number
-                       std::string datastring((char*)&data[2], datasize-2);
-                       // Throw them in an istringstream
-                       std::istringstream is(datastring, std::ios_base::binary);
-                       
-                       while(is.eof() == false)
+                       is.read(buf, 2);
+                       u16 id = readU16((u8*)buf);
+                       if(is.eof())
+                               break;
+                       is.read(buf, 2);
+                       size_t message_size = readU16((u8*)buf);
+                       std::string message;
+                       message.reserve(message_size);
+                       for(unsigned int i=0; i<message_size; i++)
                        {
-                               // Read stuff
-                               is.read(buf, 2);
-                               u16 id = readU16((u8*)buf);
-                               if(is.eof())
-                                       break;
-                               is.read(buf, 2);
-                               u16 message_size = readU16((u8*)buf);
-                               std::string message;
-                               message.reserve(message_size);
-                               for(u16 i=0; i<message_size; i++)
-                               {
-                                       is.read(buf, 1);
-                                       message.append(buf, 1);
-                               }
-                               // Pass on to the environment
-                               {
-                                       //JMutexAutoLock envlock(m_env_mutex); //bulk comment-out
-                                       m_env.processActiveObjectMessage(id, message);
-                               }
+                               is.read(buf, 1);
+                               message.append(buf, 1);
                        }
+                       // Pass on to the environment
+                       m_env.processActiveObjectMessage(id, message);
                }
        }
        else if(command == TOCLIENT_MOVEMENT)
        {
                std::string datastring((char*)&data[2], datasize-2);
                std::istringstream is(datastring, std::ios_base::binary);
-               Player *player = m_env.getLocalPlayer();
-               assert(player != NULL);
 
-               player->movement_acceleration_default = readF1000(is) * BS;
-               player->movement_acceleration_air = readF1000(is) * BS;
-               player->movement_acceleration_fast = readF1000(is) * BS;
-               player->movement_speed_walk = readF1000(is) * BS;
-               player->movement_speed_crouch = readF1000(is) * BS;
-               player->movement_speed_fast = readF1000(is) * BS;
-               player->movement_speed_climb = readF1000(is) * BS;
-               player->movement_speed_jump = readF1000(is) * BS;
-               player->movement_liquid_fluidity = readF1000(is) * BS;
+               player->movement_acceleration_default   = readF1000(is) * BS;
+               player->movement_acceleration_air       = readF1000(is) * BS;
+               player->movement_acceleration_fast      = readF1000(is) * BS;
+               player->movement_speed_walk             = readF1000(is) * BS;
+               player->movement_speed_crouch           = readF1000(is) * BS;
+               player->movement_speed_fast             = readF1000(is) * BS;
+               player->movement_speed_climb            = readF1000(is) * BS;
+               player->movement_speed_jump             = readF1000(is) * BS;
+               player->movement_liquid_fluidity        = readF1000(is) * BS;
                player->movement_liquid_fluidity_smooth = readF1000(is) * BS;
-               player->movement_liquid_sink = readF1000(is) * BS;
-               player->movement_gravity = readF1000(is) * BS;
+               player->movement_liquid_sink            = readF1000(is) * BS;
+               player->movement_gravity                = readF1000(is) * BS;
        }
        else if(command == TOCLIENT_HP)
        {
                std::string datastring((char*)&data[2], datasize-2);
                std::istringstream is(datastring, std::ios_base::binary);
-               Player *player = m_env.getLocalPlayer();
-               assert(player != NULL);
-               u8 oldhp = player->hp;
-               u8 hp = readU8(is);
+
+               u8 oldhp   = player->hp;
+               u8 hp      = readU8(is);
                player->hp = hp;
 
                if(hp < oldhp)
@@ -1502,23 +1427,18 @@ void Client::ProcessData(u8 *data, u32 datasize, u16 sender_peer_id)
        {
                std::string datastring((char*)&data[2], datasize-2);
                std::istringstream is(datastring, std::ios_base::binary);
-               Player *player = m_env.getLocalPlayer();
-               assert(player != NULL);
-               u16 breath = readU16(is);
-               player->setBreath(breath) ;
+
+               player->setBreath(readU16(is));
        }
        else if(command == TOCLIENT_MOVE_PLAYER)
        {
                std::string datastring((char*)&data[2], datasize-2);
                std::istringstream is(datastring, std::ios_base::binary);
-               Player *player = m_env.getLocalPlayer();
-               assert(player != NULL);
+
                v3f pos = readV3F1000(is);
                f32 pitch = readF1000(is);
                f32 yaw = readF1000(is);
                player->setPosition(pos);
-               /*player->setPitch(pitch);
-               player->setYaw(yaw);*/
 
                infostream<<"Client got TOCLIENT_MOVE_PLAYER"
                                <<" pos=("<<pos.X<<","<<pos.Y<<","<<pos.Z<<")"
@@ -1550,16 +1470,16 @@ void Client::ProcessData(u8 *data, u32 datasize, u16 sender_peer_id)
        {
                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.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;
+               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 if(command == TOCLIENT_ANNOUNCE_MEDIA)
@@ -1568,7 +1488,7 @@ void Client::ProcessData(u8 *data, u32 datasize, u16 sender_peer_id)
                std::istringstream is(datastring, std::ios_base::binary);
 
                int num_files = readU16(is);
-               
+
                infostream<<"Client: Received media announcement: packet size: "
                                <<datasize<<std::endl;
 
@@ -1605,16 +1525,11 @@ void Client::ProcessData(u8 *data, u32 datasize, u16 sender_peer_id)
                                        m_media_downloader->addRemoteServer(baseurl);
                        }
                }
-               catch(SerializationError) {
+               catch(SerializationError& e) {
                        // not supported by server or turned off
                }
 
                m_media_downloader->step(this);
-               if (m_media_downloader->isDone()) {
-                       // might be done already if all media is in the cache
-                       delete m_media_downloader;
-                       m_media_downloader = NULL;
-               }
        }
        else if(command == TOCLIENT_MEDIA)
        {
@@ -1660,17 +1575,12 @@ void Client::ProcessData(u8 *data, u32 datasize, u16 sender_peer_id)
                // updating content definitions
                assert(!m_mesh_update_thread.IsRunning());
 
-               for(u32 i=0; i<num_files; i++){
+               for(unsigned int i=0; i<num_files; i++){
                        std::string name = deSerializeString(is);
                        std::string data = deSerializeLongString(is);
                        m_media_downloader->conventionalTransferDone(
                                        name, data, this);
                }
-
-               if (m_media_downloader->isDone()) {
-                       delete m_media_downloader;
-                       m_media_downloader = NULL;
-               }
        }
        else if(command == TOCLIENT_TOOLDEF)
        {
@@ -1777,11 +1687,11 @@ void Client::ProcessData(u8 *data, u32 datasize, u16 sender_peer_id)
        {
                std::string datastring((char*)&data[2], datasize-2);
                std::istringstream is(datastring, std::ios_base::binary);
-               
+
                m_privileges.clear();
                infostream<<"Client: Privileges updated: ";
                u16 num_privileges = readU16(is);
-               for(u16 i=0; i<num_privileges; i++){
+               for(unsigned int i=0; i<num_privileges; i++){
                        std::string priv = deSerializeString(is);
                        m_privileges.insert(priv);
                        infostream<<priv<<" ";
@@ -1794,8 +1704,6 @@ void Client::ProcessData(u8 *data, u32 datasize, u16 sender_peer_id)
                std::istringstream is(datastring, std::ios_base::binary);
 
                // Store formspec in LocalPlayer
-               Player *player = m_env.getLocalPlayer();
-               assert(player != NULL);
                player->inventory_formspec = deSerializeLongString(is);
        }
        else if(command == TOCLIENT_DETACHED_INVENTORY)
@@ -1804,7 +1712,7 @@ void Client::ProcessData(u8 *data, u32 datasize, u16 sender_peer_id)
                std::istringstream is(datastring, std::ios_base::binary);
 
                std::string name = deSerializeString(is);
-               
+
                infostream<<"Client: Detached inventory update: \""<<name<<"\""<<std::endl;
 
                Inventory *inv = NULL;
@@ -1837,30 +1745,28 @@ void Client::ProcessData(u8 *data, u32 datasize, u16 sender_peer_id)
                std::string datastring((char*)&data[2], datasize-2);
                std::istringstream is(datastring, std::ios_base::binary);
 
-               v3f pos = readV3F1000(is);
-               v3f vel = readV3F1000(is);
-               v3f acc = readV3F1000(is);
-               float expirationtime = readF1000(is);
-               float size = readF1000(is);
+               v3f pos                 = readV3F1000(is);
+               v3f vel                 = readV3F1000(is);
+               v3f acc                 = readV3F1000(is);
+               float expirationtime    = readF1000(is);
+               float size              = readF1000(is);
                bool collisiondetection = readU8(is);
-               std::string texture = deSerializeLongString(is);
-               bool vertical = false;
+               std::string texture     = deSerializeLongString(is);
+               bool vertical           = false;
                try {
                        vertical = readU8(is);
                } catch (...) {}
 
                ClientEvent event;
-               event.type = CE_SPAWN_PARTICLE;
-               event.spawn_particle.pos = new v3f (pos);
-               event.spawn_particle.vel = new v3f (vel);
-               event.spawn_particle.acc = new v3f (acc);
-
-               event.spawn_particle.expirationtime = expirationtime;
-               event.spawn_particle.size = size;
-               event.spawn_particle.collisiondetection =
-                               collisiondetection;
-               event.spawn_particle.vertical = vertical;
-               event.spawn_particle.texture = new std::string(texture);
+               event.type                              = CE_SPAWN_PARTICLE;
+               event.spawn_particle.pos                = new v3f (pos);
+               event.spawn_particle.vel                = new v3f (vel);
+               event.spawn_particle.acc                = new v3f (acc);
+               event.spawn_particle.expirationtime     = expirationtime;
+               event.spawn_particle.size               = size;
+               event.spawn_particle.collisiondetection = collisiondetection;
+               event.spawn_particle.vertical           = vertical;
+               event.spawn_particle.texture            = new std::string(texture);
 
                m_client_event_queue.push_back(event);
        }
@@ -1869,46 +1775,44 @@ void Client::ProcessData(u8 *data, u32 datasize, u16 sender_peer_id)
                std::string datastring((char*)&data[2], datasize-2);
                std::istringstream is(datastring, std::ios_base::binary);
 
-               u16 amount = readU16(is);
-               float spawntime = readF1000(is);
-               v3f minpos = readV3F1000(is);
-               v3f maxpos = readV3F1000(is);
-               v3f minvel = readV3F1000(is);
-               v3f maxvel = readV3F1000(is);
-               v3f minacc = readV3F1000(is);
-               v3f maxacc = readV3F1000(is);
-               float minexptime = readF1000(is);
-               float maxexptime = readF1000(is);
-               float minsize = readF1000(is);
-               float maxsize = readF1000(is);
+               u16 amount              = readU16(is);
+               float spawntime         = readF1000(is);
+               v3f minpos              = readV3F1000(is);
+               v3f maxpos              = readV3F1000(is);
+               v3f minvel              = readV3F1000(is);
+               v3f maxvel              = readV3F1000(is);
+               v3f minacc              = readV3F1000(is);
+               v3f maxacc              = readV3F1000(is);
+               float minexptime        = readF1000(is);
+               float maxexptime        = readF1000(is);
+               float minsize           = readF1000(is);
+               float maxsize           = readF1000(is);
                bool collisiondetection = readU8(is);
-               std::string texture = deSerializeLongString(is);
-               u32 id = readU32(is);
+               std::string texture     = deSerializeLongString(is);
+               u32 id                  = readU32(is);
                bool vertical = false;
                try {
                        vertical = readU8(is);
                } catch (...) {}
 
                ClientEvent event;
-               event.type = CE_ADD_PARTICLESPAWNER;
-               event.add_particlespawner.amount = amount;
-               event.add_particlespawner.spawntime = spawntime;
-
-               event.add_particlespawner.minpos = new v3f (minpos);
-               event.add_particlespawner.maxpos = new v3f (maxpos);
-               event.add_particlespawner.minvel = new v3f (minvel);
-               event.add_particlespawner.maxvel = new v3f (maxvel);
-               event.add_particlespawner.minacc = new v3f (minacc);
-               event.add_particlespawner.maxacc = new v3f (maxacc);
-
-               event.add_particlespawner.minexptime = minexptime;
-               event.add_particlespawner.maxexptime = maxexptime;
-               event.add_particlespawner.minsize = minsize;
-               event.add_particlespawner.maxsize = maxsize;
+               event.type                                   = CE_ADD_PARTICLESPAWNER;
+               event.add_particlespawner.amount             = amount;
+               event.add_particlespawner.spawntime          = spawntime;
+               event.add_particlespawner.minpos             = new v3f (minpos);
+               event.add_particlespawner.maxpos             = new v3f (maxpos);
+               event.add_particlespawner.minvel             = new v3f (minvel);
+               event.add_particlespawner.maxvel             = new v3f (maxvel);
+               event.add_particlespawner.minacc             = new v3f (minacc);
+               event.add_particlespawner.maxacc             = new v3f (maxacc);
+               event.add_particlespawner.minexptime         = minexptime;
+               event.add_particlespawner.maxexptime         = maxexptime;
+               event.add_particlespawner.minsize            = minsize;
+               event.add_particlespawner.maxsize            = maxsize;
                event.add_particlespawner.collisiondetection = collisiondetection;
-               event.add_particlespawner.vertical = vertical;
-               event.add_particlespawner.texture = new std::string(texture);
-               event.add_particlespawner.id = id;
+               event.add_particlespawner.vertical           = vertical;
+               event.add_particlespawner.texture            = new std::string(texture);
+               event.add_particlespawner.id                 = id;
 
                m_client_event_queue.push_back(event);
        }
@@ -1920,7 +1824,7 @@ void Client::ProcessData(u8 *data, u32 datasize, u16 sender_peer_id)
                u32 id = readU16(is);
 
                ClientEvent event;
-               event.type = CE_DELETE_PARTICLESPAWNER;
+               event.type                      = CE_DELETE_PARTICLESPAWNER;
                event.delete_particlespawner.id = id;
 
                m_client_event_queue.push_back(event);
@@ -1941,20 +1845,30 @@ void Client::ProcessData(u8 *data, u32 datasize, u16 sender_peer_id)
                u32 dir          = readU32(is);
                v2f align        = readV2F1000(is);
                v2f offset       = readV2F1000(is);
+               v3f world_pos;
+               v2s32 size;
+               try{
+                       world_pos    = readV3F1000(is);
+               }catch(SerializationError &e) {};
+               try{
+                       size = readV2S32(is);
+               } catch(SerializationError &e) {};
 
                ClientEvent event;
-               event.type = CE_HUDADD;
-               event.hudadd.id     = id;
-               event.hudadd.type   = type;
-               event.hudadd.pos    = new v2f(pos);
-               event.hudadd.name   = new std::string(name);
-               event.hudadd.scale  = new v2f(scale);
-               event.hudadd.text   = new std::string(text);
-               event.hudadd.number = number;
-               event.hudadd.item   = item;
-               event.hudadd.dir    = dir;
-               event.hudadd.align  = new v2f(align);
-               event.hudadd.offset = new v2f(offset);
+               event.type             = CE_HUDADD;
+               event.hudadd.id        = id;
+               event.hudadd.type      = type;
+               event.hudadd.pos       = new v2f(pos);
+               event.hudadd.name      = new std::string(name);
+               event.hudadd.scale     = new v2f(scale);
+               event.hudadd.text      = new std::string(text);
+               event.hudadd.number    = number;
+               event.hudadd.item      = item;
+               event.hudadd.dir       = dir;
+               event.hudadd.align     = new v2f(align);
+               event.hudadd.offset    = new v2f(offset);
+               event.hudadd.world_pos = new v3f(world_pos);
+               event.hudadd.size      = new v2s32(size);
                m_client_event_queue.push_back(event);
        }
        else if(command == TOCLIENT_HUDRM)
@@ -1965,7 +1879,7 @@ void Client::ProcessData(u8 *data, u32 datasize, u16 sender_peer_id)
                u32 id = readU32(is);
 
                ClientEvent event;
-               event.type = CE_HUDRM;
+               event.type     = CE_HUDRM;
                event.hudrm.id = id;
                m_client_event_queue.push_back(event);
        }
@@ -1973,29 +1887,37 @@ void Client::ProcessData(u8 *data, u32 datasize, u16 sender_peer_id)
        {
                std::string sdata;
                v2f v2fdata;
+               v3f v3fdata;
                u32 intdata = 0;
-               
+               v2s32 v2s32data;
+
                std::string datastring((char *)&data[2], datasize - 2);
                std::istringstream is(datastring, std::ios_base::binary);
 
                u32 id  = readU32(is);
                u8 stat = (HudElementStat)readU8(is);
-               
+
                if (stat == HUD_STAT_POS || stat == HUD_STAT_SCALE ||
                        stat == HUD_STAT_ALIGN || stat == HUD_STAT_OFFSET)
                        v2fdata = readV2F1000(is);
                else if (stat == HUD_STAT_NAME || stat == HUD_STAT_TEXT)
                        sdata = deSerializeString(is);
+               else if (stat == HUD_STAT_WORLD_POS)
+                       v3fdata = readV3F1000(is);
+               else if (stat == HUD_STAT_SIZE )
+                       v2s32data = readV2S32(is);
                else
                        intdata = readU32(is);
-               
+
                ClientEvent event;
-               event.type = CE_HUDCHANGE;
+               event.type              = CE_HUDCHANGE;
                event.hudchange.id      = id;
                event.hudchange.stat    = (HudElementStat)stat;
                event.hudchange.v2fdata = new v2f(v2fdata);
+               event.hudchange.v3fdata = new v3f(v3fdata);
                event.hudchange.sdata   = new std::string(sdata);
                event.hudchange.data    = intdata;
+               event.hudchange.v2s32data = new v2s32(v2s32data);
                m_client_event_queue.push_back(event);
        }
        else if(command == TOCLIENT_HUD_SET_FLAGS)
@@ -2003,12 +1925,9 @@ void Client::ProcessData(u8 *data, u32 datasize, u16 sender_peer_id)
                std::string datastring((char *)&data[2], datasize - 2);
                std::istringstream is(datastring, std::ios_base::binary);
 
-               Player *player = m_env.getLocalPlayer();
-               assert(player != NULL);
-
                u32 flags = readU32(is);
                u32 mask  = readU32(is);
-               
+
                player->hud_flags &= ~mask;
                player->hud_flags |= flags;
        }
@@ -2017,22 +1936,80 @@ void Client::ProcessData(u8 *data, u32 datasize, u16 sender_peer_id)
                std::string datastring((char *)&data[2], datasize - 2);
                std::istringstream is(datastring, std::ios_base::binary);
 
-               Player *player = m_env.getLocalPlayer();
-               assert(player != NULL);
-
                u16 param         = readU16(is);
                std::string value = deSerializeString(is);
 
-               if(param == HUD_PARAM_HOTBAR_ITEMCOUNT && value.size() == 4){
+               if(param == HUD_PARAM_HOTBAR_ITEMCOUNT && value.size() == 4) {
                        s32 hotbar_itemcount = readS32((u8*) value.c_str());
                        if(hotbar_itemcount > 0 && hotbar_itemcount <= HUD_HOTBAR_ITEMCOUNT_MAX)
                                player->hud_hotbar_itemcount = hotbar_itemcount;
-               } else if (param == HUD_PARAM_HOTBAR_IMAGE) {
+               }
+               else if (param == HUD_PARAM_HOTBAR_IMAGE) {
                        ((LocalPlayer *) player)->hotbar_image = value;
-               } else if (param == HUD_PARAM_HOTBAR_SELECTED_IMAGE) {
+               }
+               else if (param == HUD_PARAM_HOTBAR_SELECTED_IMAGE) {
                        ((LocalPlayer *) player)->hotbar_selected_image = value;
                }
        }
+       else if(command == TOCLIENT_SET_SKY)
+       {
+               std::string datastring((char *)&data[2], datasize - 2);
+               std::istringstream is(datastring, std::ios_base::binary);
+
+               video::SColor *bgcolor           = new video::SColor(readARGB8(is));
+               std::string *type                = new std::string(deSerializeString(is));
+               u16 count                        = readU16(is);
+               std::vector<std::string> *params = new std::vector<std::string>;
+
+               for(size_t i=0; i<count; i++)
+                       params->push_back(deSerializeString(is));
+
+               ClientEvent event;
+               event.type            = CE_SET_SKY;
+               event.set_sky.bgcolor = bgcolor;
+               event.set_sky.type    = type;
+               event.set_sky.params  = params;
+               m_client_event_queue.push_back(event);
+       }
+       else if(command == TOCLIENT_OVERRIDE_DAY_NIGHT_RATIO)
+       {
+               std::string datastring((char *)&data[2], datasize - 2);
+               std::istringstream is(datastring, std::ios_base::binary);
+
+               bool do_override        = readU8(is);
+               float day_night_ratio_f = (float)readU16(is) / 65536;
+
+               ClientEvent event;
+               event.type                                 = CE_OVERRIDE_DAY_NIGHT_RATIO;
+               event.override_day_night_ratio.do_override = do_override;
+               event.override_day_night_ratio.ratio_f     = day_night_ratio_f;
+               m_client_event_queue.push_back(event);
+       }
+       else if(command == TOCLIENT_LOCAL_PLAYER_ANIMATIONS)
+       {
+               std::string datastring((char *)&data[2], datasize - 2);
+               std::istringstream is(datastring, std::ios_base::binary);
+
+               LocalPlayer *player = m_env.getLocalPlayer();
+               assert(player != NULL);
+
+               player->local_animations[0] = readV2S32(is);
+               player->local_animations[1] = readV2S32(is);
+               player->local_animations[2] = readV2S32(is);
+               player->local_animations[3] = readV2S32(is);
+               player->local_animation_speed = readF1000(is);
+       }
+       else if(command == TOCLIENT_EYE_OFFSET)
+       {
+               std::string datastring((char *)&data[2], datasize - 2);
+               std::istringstream is(datastring, std::ios_base::binary);
+
+               LocalPlayer *player = m_env.getLocalPlayer();
+               assert(player != NULL);
+
+               player->eye_offset_first = readV3F1000(is);
+               player->eye_offset_third = readV3F1000(is);
+       }
        else
        {
                infostream<<"Client: Ignoring unknown command "
@@ -2048,7 +2025,7 @@ void Client::Send(u16 channelnum, SharedBuffer<u8> data, bool reliable)
 
 void Client::interact(u8 action, const PointedThing& pointed)
 {
-       if(connectedAndInitialized() == false){
+       if(m_state != LC_Ready){
                infostream<<"Client::interact() "
                                "cancelled (not connected)"
                                <<std::endl;
@@ -2092,7 +2069,9 @@ void Client::sendNodemetaFields(v3s16 p, const std::string &formname,
        writeU16(os, TOSERVER_NODEMETA_FIELDS);
        writeV3S16(os, p);
        os<<serializeString(formname);
-       writeU16(os, fields.size());
+       size_t fields_size = fields.size();
+       assert(fields_size <= 0xFFFF);
+       writeU16(os, (u16) (fields_size & 0xFFFF));
        for(std::map<std::string, std::string>::const_iterator
                        i = fields.begin(); i != fields.end(); i++){
                const std::string &name = i->first;
@@ -2107,7 +2086,7 @@ void Client::sendNodemetaFields(v3s16 p, const std::string &formname,
        // Send as reliable
        Send(0, data, true);
 }
-       
+
 void Client::sendInventoryFields(const std::string &formname,
                const std::map<std::string, std::string> &fields)
 {
@@ -2115,10 +2094,12 @@ void Client::sendInventoryFields(const std::string &formname,
 
        writeU16(os, TOSERVER_INVENTORY_FIELDS);
        os<<serializeString(formname);
-       writeU16(os, fields.size());
+       size_t fields_size = fields.size();
+       assert(fields_size <= 0xFFFF);
+       writeU16(os, (u16) (fields_size & 0xFFFF));
        for(std::map<std::string, std::string>::const_iterator
                        i = fields.begin(); i != fields.end(); i++){
-               const std::string &name = i->first;
+               const std::string &name  = i->first;
                const std::string &value = i->second;
                os<<serializeString(name);
                os<<serializeLongString(value);
@@ -2135,13 +2116,13 @@ void Client::sendInventoryAction(InventoryAction *a)
 {
        std::ostringstream os(std::ios_base::binary);
        u8 buf[12];
-       
+
        // Write command
        writeU16(buf, TOSERVER_INVENTORY_ACTION);
        os.write((char*)buf, 2);
 
        a->serialize(os);
-       
+
        // Make data buffer
        std::string s = os.str();
        SharedBuffer<u8> data((u8*)s.c_str(), s.size());
@@ -2153,23 +2134,27 @@ void Client::sendChatMessage(const std::wstring &message)
 {
        std::ostringstream os(std::ios_base::binary);
        u8 buf[12];
-       
+
        // Write command
        writeU16(buf, TOSERVER_CHAT_MESSAGE);
        os.write((char*)buf, 2);
-       
+
        // Write length
-       writeU16(buf, message.size());
+       size_t messagesize = message.size();
+       if (messagesize > 0xFFFF) {
+               messagesize = 0xFFFF;
+       }
+       writeU16(buf, (u16) messagesize);
        os.write((char*)buf, 2);
-       
+
        // Write string
-       for(u32 i=0; i<message.size(); i++)
+       for(unsigned int i=0; i<message.size(); i++)
        {
                u16 w = message[i];
                writeU16(buf, w);
                os.write((char*)buf, 2);
        }
-       
+
        // Make data buffer
        std::string s = os.str();
        SharedBuffer<u8> data((u8*)s.c_str(), s.size());
@@ -2177,8 +2162,8 @@ void Client::sendChatMessage(const std::wstring &message)
        Send(0, data, true);
 }
 
-void Client::sendChangePassword(const std::wstring oldpassword,
-               const std::wstring newpassword)
+void Client::sendChangePassword(const std::wstring &oldpassword,
+                                const std::wstring &newpassword)
 {
        Player *player = m_env.getLocalPlayer();
        if(player == NULL)
@@ -2197,7 +2182,7 @@ void Client::sendChangePassword(const std::wstring oldpassword,
        */
 
        writeU16(buf, TOSERVER_PASSWORD);
-       for(u32 i=0;i<PASSWORD_SIZE-1;i++)
+       for(unsigned int 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;
@@ -2257,26 +2242,45 @@ void Client::sendRespawn()
        Send(0, data, true);
 }
 
+void Client::sendReady()
+{
+       DSTACK(__FUNCTION_NAME);
+       std::ostringstream os(std::ios_base::binary);
+
+       writeU16(os, TOSERVER_CLIENT_READY);
+       writeU8(os,VERSION_MAJOR);
+       writeU8(os,VERSION_MINOR);
+       writeU8(os,VERSION_PATCH_ORIG);
+       writeU8(os,0);
+
+       writeU16(os,strlen(minetest_version_hash));
+       os.write(minetest_version_hash,strlen(minetest_version_hash));
+
+       // 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
-       
        LocalPlayer *myplayer = m_env.getLocalPlayer();
        if(myplayer == NULL)
                return;
 
        // Save bandwidth by only updating position when something changed
-       if(myplayer->last_position == myplayer->getPosition() &&
-                       myplayer->last_speed == myplayer->getSpeed() &&
-                       myplayer->last_pitch == myplayer->getPitch() &&
-                       myplayer->last_yaw == myplayer->getYaw() &&
+       if(myplayer->last_position        == myplayer->getPosition() &&
+                       myplayer->last_speed      == myplayer->getSpeed()    &&
+                       myplayer->last_pitch      == myplayer->getPitch()    &&
+                       myplayer->last_yaw        == myplayer->getYaw()      &&
                        myplayer->last_keyPressed == myplayer->keyPressed)
                return;
 
-       myplayer->last_position = myplayer->getPosition();
-       myplayer->last_speed = myplayer->getSpeed();
-       myplayer->last_pitch = myplayer->getPitch();
-       myplayer->last_yaw = myplayer->getYaw();
+       myplayer->last_position   = myplayer->getPosition();
+       myplayer->last_speed      = myplayer->getSpeed();
+       myplayer->last_pitch      = myplayer->getPitch();
+       myplayer->last_yaw        = myplayer->getYaw();
        myplayer->last_keyPressed = myplayer->keyPressed;
 
        u16 our_peer_id;
@@ -2284,20 +2288,21 @@ void Client::sendPlayerPos()
                //JMutexAutoLock lock(m_con_mutex); //bulk comment-out
                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);
-       
-       v3f pf = myplayer->getPosition();
+
+       v3f pf         = myplayer->getPosition();
+       v3f sf         = myplayer->getSpeed();
+       s32 pitch      = myplayer->getPitch() * 100;
+       s32 yaw        = myplayer->getYaw() * 100;
+       u32 keyPressed = myplayer->keyPressed;
+
        v3s32 position(pf.X*100, pf.Y*100, pf.Z*100);
-       v3f sf = myplayer->getSpeed();
        v3s32 speed(sf.X*100, sf.Y*100, sf.Z*100);
-       s32 pitch = myplayer->getPitch() * 100;
-       s32 yaw = myplayer->getYaw() * 100;
-       u32 keyPressed=myplayer->keyPressed;
        /*
                Format:
                [0] u16 command
@@ -2346,27 +2351,23 @@ void Client::removeNode(v3s16 p)
 
        try
        {
-               //TimeTaker t("removeNodeAndUpdate", m_device);
                m_env.getMap().removeNodeAndUpdate(p, modified_blocks);
        }
        catch(InvalidPositionException &e)
        {
        }
-       
-       // add urgent task to update the modified node
-       addUpdateMeshTaskForNode(p, false, true);
 
        for(std::map<v3s16, MapBlock * >::iterator
                        i = modified_blocks.begin();
                        i != modified_blocks.end(); ++i)
        {
-               addUpdateMeshTaskWithEdge(i->first);
+               addUpdateMeshTaskWithEdge(i->first, false, true);
        }
 }
 
 void Client::addNode(v3s16 p, MapNode n, bool remove_metadata)
 {
-       TimeTaker timer1("Client::addNode()");
+       //TimeTaker timer1("Client::addNode()");
 
        std::map<v3s16, MapBlock*> modified_blocks;
 
@@ -2377,18 +2378,17 @@ void Client::addNode(v3s16 p, MapNode n, bool remove_metadata)
        }
        catch(InvalidPositionException &e)
        {}
-       
+
        for(std::map<v3s16, MapBlock * >::iterator
                        i = modified_blocks.begin();
                        i != modified_blocks.end(); ++i)
        {
-               addUpdateMeshTaskWithEdge(i->first);
+               addUpdateMeshTaskWithEdge(i->first, false, true);
        }
 }
-       
+
 void Client::setPlayerControl(PlayerControl &control)
 {
-       //JMutexAutoLock envlock(m_env_mutex); //bulk comment-out
        LocalPlayer *player = m_env.getLocalPlayer();
        assert(player != NULL);
        player->control = control;
@@ -2396,7 +2396,6 @@ void Client::setPlayerControl(PlayerControl &control)
 
 void Client::selectPlayerItem(u16 item)
 {
-       //JMutexAutoLock envlock(m_env_mutex); //bulk comment-out
        m_playeritem = item;
        m_inventory_updated = true;
        sendPlayerItem(item);
@@ -2406,8 +2405,6 @@ void Client::selectPlayerItem(u16 item)
 // updated from the server. If it is true, it is set to false.
 bool Client::getLocalInventoryUpdated()
 {
-       // m_inventory_updated is behind envlock
-       //JMutexAutoLock envlock(m_env_mutex); //bulk comment-out
        bool updated = m_inventory_updated;
        m_inventory_updated = false;
        return updated;
@@ -2416,7 +2413,6 @@ bool Client::getLocalInventoryUpdated()
 // Copies the inventory of the local player to parameter
 void Client::getLocalInventory(Inventory &dst)
 {
-       //JMutexAutoLock envlock(m_env_mutex); //bulk comment-out
        Player *player = m_env.getLocalPlayer();
        assert(player != NULL);
        dst = player->inventory;
@@ -2463,6 +2459,7 @@ Inventory* Client::getInventory(const InventoryLocation &loc)
        }
        return NULL;
 }
+
 void Client::inventoryAction(InventoryAction *a)
 {
        /*
@@ -2489,16 +2486,14 @@ ClientActiveObject * Client::getSelectedActiveObject(
 
        m_env.getActiveObjects(from_pos_f_on_map, max_d, objects);
 
-       //infostream<<"Collected "<<objects.size()<<" nearby objects"<<std::endl;
-       
        // Sort them.
        // After this, the closest object is the first in the array.
        std::sort(objects.begin(), objects.end());
 
-       for(u32 i=0; i<objects.size(); i++)
+       for(unsigned int i=0; i<objects.size(); i++)
        {
                ClientActiveObject *obj = objects[i].obj;
-               
+
                core::aabbox3d<f32> *selection_box = obj->getSelectionBox();
                if(selection_box == NULL)
                        continue;
@@ -2512,26 +2507,13 @@ ClientActiveObject * Client::getSelectedActiveObject(
 
                if(offsetted_box.intersectsWithLine(shootline_on_map))
                {
-                       //infostream<<"Returning selected object"<<std::endl;
                        return obj;
                }
        }
 
-       //infostream<<"No object selected; returning NULL."<<std::endl;
        return NULL;
 }
 
-void Client::printDebugInfo(std::ostream &os)
-{
-       //JMutexAutoLock lock1(m_fetchblock_mutex);
-       /*JMutexAutoLock lock2(m_incoming_queue_mutex);
-
-       os<<"m_incoming_queue.getSize()="<<m_incoming_queue.getSize()
-               //<<", m_fetchblock_history.size()="<<m_fetchblock_history.size()
-               //<<", m_opt_not_found_history.size()="<<m_opt_not_found_history.size()
-               <<std::endl;*/
-}
-
 std::list<std::string> Client::getConnectedPlayerNames()
 {
        return m_env.getPlayerNames();
@@ -2547,6 +2529,15 @@ int Client::getCrackLevel()
        return m_crack_level;
 }
 
+void Client::setHighlighted(v3s16 pos, bool show_highlighted)
+{
+       m_show_highlighted = show_highlighted;
+       v3s16 old_highlighted_pos = m_highlighted_pos;
+       m_highlighted_pos = pos;
+       addUpdateMeshTaskForNode(old_highlighted_pos, false, true);
+       addUpdateMeshTaskForNode(m_highlighted_pos, false, true);
+}
+
 void Client::setCrack(int level, v3s16 pos)
 {
        int old_crack_level = m_crack_level;
@@ -2601,72 +2592,50 @@ void Client::typeChatMessage(const std::wstring &message)
        // Show locally
        if (message[0] == L'/')
        {
-               m_chat_queue.push_back(
-                               (std::wstring)L"issued command: "+message);
+               m_chat_queue.push_back((std::wstring)L"issued command: " + message);
        }
        else
        {
                LocalPlayer *player = m_env.getLocalPlayer();
                assert(player != NULL);
                std::wstring name = narrow_to_wide(player->getName());
-               m_chat_queue.push_back(
-                               (std::wstring)L"<"+name+L"> "+message);
+               m_chat_queue.push_back((std::wstring)L"<" + name + L"> " + message);
        }
 }
 
 void Client::addUpdateMeshTask(v3s16 p, bool ack_to_server, bool urgent)
 {
-       /*infostream<<"Client::addUpdateMeshTask(): "
-                       <<"("<<p.X<<","<<p.Y<<","<<p.Z<<")"
-                       <<" ack_to_server="<<ack_to_server
-                       <<" urgent="<<urgent
-                       <<std::endl;*/
-
        MapBlock *b = m_env.getMap().getBlockNoCreateNoEx(p);
        if(b == NULL)
                return;
-       
+
        /*
                Create a task to update the mesh of the block
        */
-       
+
        MeshMakeData *data = new MeshMakeData(this);
-       
+
        {
                //TimeTaker timer("data fill");
                // Release: ~0ms
                // Debug: 1-6ms, avg=2ms
                data->fill(b);
                data->setCrack(m_crack_level, m_crack_pos);
-               data->setSmoothLighting(g_settings->getBool("smooth_lighting"));
+               data->setHighlighted(m_highlighted_pos, m_show_highlighted);
+               data->setSmoothLighting(m_cache_smooth_lighting);
        }
 
-       // Debug wait
-       //while(m_mesh_update_thread.m_queue_in.size() > 0) sleep_ms(10);
-       
        // Add task to queue
        m_mesh_update_thread.m_queue_in.addBlock(p, data, ack_to_server, urgent);
-
-       /*infostream<<"Mesh update input queue size is "
-                       <<m_mesh_update_thread.m_queue_in.size()
-                       <<std::endl;*/
 }
 
 void Client::addUpdateMeshTaskWithEdge(v3s16 blockpos, bool ack_to_server, bool urgent)
 {
-       /*{
-               v3s16 p = blockpos;
-               infostream<<"Client::addUpdateMeshTaskWithEdge(): "
-                               <<"("<<p.X<<","<<p.Y<<","<<p.Z<<")"
-                               <<std::endl;
-       }*/
-
        try{
-               v3s16 p = blockpos + v3s16(0,0,0);
-               //MapBlock *b = m_env.getMap().getBlockNoCreate(p);
-               addUpdateMeshTask(p, ack_to_server, urgent);
+               addUpdateMeshTask(blockpos, ack_to_server, urgent);
        }
        catch(InvalidPositionException &e){}
+
        // Leading edge
        for (int i=0;i<6;i++)
        {
@@ -2687,14 +2656,14 @@ void Client::addUpdateMeshTaskForNode(v3s16 nodepos, bool ack_to_server, bool ur
                                <<std::endl;
        }
 
-       v3s16 blockpos = getNodeBlockPos(nodepos);
+       v3s16 blockpos          = getNodeBlockPos(nodepos);
        v3s16 blockpos_relative = blockpos * MAP_BLOCKSIZE;
 
        try{
-               v3s16 p = blockpos + v3s16(0,0,0);
-               addUpdateMeshTask(p, ack_to_server, urgent);
+               addUpdateMeshTask(blockpos, ack_to_server, urgent);
        }
        catch(InvalidPositionException &e){}
+
        // Leading edge
        if(nodepos.X == blockpos_relative.X){
                try{
@@ -2703,6 +2672,7 @@ void Client::addUpdateMeshTaskForNode(v3s16 nodepos, bool ack_to_server, bool ur
                }
                catch(InvalidPositionException &e){}
        }
+
        if(nodepos.Y == blockpos_relative.Y){
                try{
                        v3s16 p = blockpos + v3s16(0,-1,0);
@@ -2710,6 +2680,7 @@ void Client::addUpdateMeshTaskForNode(v3s16 nodepos, bool ack_to_server, bool ur
                }
                catch(InvalidPositionException &e){}
        }
+
        if(nodepos.Z == blockpos_relative.Z){
                try{
                        v3s16 p = blockpos + v3s16(0,0,-1);
@@ -2738,51 +2709,60 @@ float Client::mediaReceiveProgress()
                return 1.0; // downloader only exists when not yet done
 }
 
-void draw_load_screen(const std::wstring &text,
-               IrrlichtDevice* device, gui::IGUIFont* font,
-               float dtime=0 ,int percent=0, bool clouds=true);
 void Client::afterContentReceived(IrrlichtDevice *device, gui::IGUIFont* font)
 {
        infostream<<"Client::afterContentReceived() started"<<std::endl;
        assert(m_itemdef_received);
        assert(m_nodedef_received);
        assert(mediaReceived());
-       
+
+       wchar_t* text = wgettext("Loading textures...");
+
        // Rebuild inherited images and recreate textures
        infostream<<"- Rebuilding images and textures"<<std::endl;
+       draw_load_screen(text,device, guienv, 0, 70);
        m_tsrc->rebuildImagesAndTextures();
+       delete[] text;
 
        // Rebuild shaders
        infostream<<"- Rebuilding shaders"<<std::endl;
+       text = wgettext("Rebuilding shaders...");
+       draw_load_screen(text, device, guienv, 0, 75);
        m_shsrc->rebuildShaders();
+       delete[] text;
 
        // Update node aliases
        infostream<<"- Updating node aliases"<<std::endl;
+       text = wgettext("Initializing nodes...");
+       draw_load_screen(text, device, guienv, 0, 80);
        m_nodedef->updateAliases(m_itemdef);
+       m_nodedef->setNodeRegistrationStatus(true);
+       m_nodedef->runNodeResolverCallbacks();
+       delete[] text;
 
-       // Update node textures
+       // Update node textures and assign shaders to each tile
        infostream<<"- Updating node textures"<<std::endl;
-       m_nodedef->updateTextures(m_tsrc);
+       m_nodedef->updateTextures(this);
 
        // Preload item textures and meshes if configured to
        if(g_settings->getBool("preload_item_visuals"))
        {
                verbosestream<<"Updating item textures and meshes"<<std::endl;
-               wchar_t* text = wgettext("Item textures...");
-               draw_load_screen(text,device,font,0,0);
+               text = wgettext("Item textures...");
+               draw_load_screen(text, device, guienv, 0, 0);
                std::set<std::string> names = m_itemdef->getAll();
                size_t size = names.size();
                size_t count = 0;
                int percent = 0;
                for(std::set<std::string>::const_iterator
-                               i = names.begin(); i != names.end(); ++i){
+                               i = names.begin(); i != names.end(); ++i)
+               {
                        // Asking for these caches the result
                        m_itemdef->getInventoryTexture(*i, this);
                        m_itemdef->getWieldMesh(*i, this);
                        count++;
-                       percent = count*100/size;
-                       if (count%50 == 0) // only update every 50 item
-                               draw_load_screen(text,device,font,0,percent);
+                       percent = (count * 100 / size * 0.2) + 80;
+                       draw_load_screen(text, device, guienv, 0, percent);
                }
                delete[] text;
        }
@@ -2790,16 +2770,57 @@ void Client::afterContentReceived(IrrlichtDevice *device, gui::IGUIFont* font)
        // Start mesh update thread after setting up content definitions
        infostream<<"- Starting mesh update thread"<<std::endl;
        m_mesh_update_thread.Start();
-       
+
+       m_state = LC_Ready;
+       sendReady();
+       text = wgettext("Done!");
+       draw_load_screen(text, device, guienv, 0, 100);
        infostream<<"Client::afterContentReceived() done"<<std::endl;
+       delete[] text;
 }
 
 float Client::getRTT(void)
 {
-       try{
-               return m_con.GetPeerAvgRTT(PEER_ID_SERVER);
-       } catch(con::PeerNotFoundException &e){
-               return 1337;
+       return m_con.getPeerStat(PEER_ID_SERVER,con::AVG_RTT);
+}
+
+float Client::getCurRate(void)
+{
+       return ( m_con.getLocalStat(con::CUR_INC_RATE) +
+                       m_con.getLocalStat(con::CUR_DL_RATE));
+}
+
+float Client::getAvgRate(void)
+{
+       return ( m_con.getLocalStat(con::AVG_INC_RATE) +
+                       m_con.getLocalStat(con::AVG_DL_RATE));
+}
+
+void Client::makeScreenshot(IrrlichtDevice *device)
+{
+       irr::video::IVideoDriver *driver = device->getVideoDriver();
+       irr::video::IImage* const raw_image = driver->createScreenShot();
+       if (raw_image) {
+               irr::video::IImage* const image = driver->createImage(video::ECF_R8G8B8,
+                       raw_image->getDimension());
+
+               if (image) {
+                       raw_image->copyTo(image);
+                       irr::c8 filename[256];
+                       snprintf(filename, sizeof(filename), "%s" DIR_DELIM "screenshot_%u.png",
+                                g_settings->get("screenshot_path").c_str(),
+                                device->getTimer()->getRealTime());
+                       std::ostringstream sstr;
+                       if (driver->writeImageToFile(image, filename)) {
+                               sstr << "Saved screenshot to '" << filename << "'";
+                       } else {
+                               sstr << "Failed to save screenshot '" << filename << "'";
+                       }
+                       m_chat_queue.push_back(narrow_to_wide(sstr.str()));
+                       infostream << sstr.str() << std::endl;
+                       image->drop();
+               }
+               raw_image->drop();
        }
 }
 
@@ -2826,6 +2847,10 @@ IShaderSource* Client::getShaderSource()
 {
        return m_shsrc;
 }
+scene::ISceneManager* Client::getSceneManager()
+{
+       return m_device->getSceneManager();
+}
 u16 Client::allocateUnknownNodeId(const std::string &name)
 {
        errorstream<<"Client::allocateUnknownNodeId(): "
@@ -2842,6 +2867,11 @@ MtEventManager* Client::getEventManager()
        return m_event;
 }
 
+ParticleManager* Client::getParticleManager()
+{
+       return &m_particle_manager;
+}
+
 scene::IAnimatedMesh* Client::getMesh(const std::string &filename)
 {
        std::map<std::string, std::string>::const_iterator i =
@@ -2851,16 +2881,17 @@ scene::IAnimatedMesh* Client::getMesh(const std::string &filename)
                                <<std::endl;
                return NULL;
        }
-       const std::string &data = i->second;
+       const std::string &data    = i->second;
        scene::ISceneManager *smgr = m_device->getSceneManager();
 
        // Create the mesh, remove it from cache and return it
        // This allows unique vertex colors and other properties for each instance
        Buffer<char> data_rw(data.c_str(), data.size()); // Const-incorrect Irrlicht
        io::IFileSystem *irrfs = m_device->getFileSystem();
-       io::IReadFile *rfile = irrfs->createMemoryReadFile(
+       io::IReadFile *rfile   = irrfs->createMemoryReadFile(
                        *data_rw, data_rw.getSize(), filename.c_str());
        assert(rfile);
+
        scene::IAnimatedMesh *mesh = smgr->getMesh(rfile);
        rfile->drop();
        // NOTE: By playing with Irrlicht refcounts, maybe we could cache a bunch