Commented out debug statements again
[oweals/minetest.git] / src / server.cpp
index 16ad80c00ce822dab8903064c80896d8cf48fe04..fd93d7523b22f872fdbf8f2b9106c403a6382b21 100644 (file)
@@ -31,9 +31,39 @@ with this program; if not, write to the Free Software Foundation, Inc.,
 #include "config.h"
 #include "servercommand.h"
 #include "filesys.h"
+#include "content_mapnode.h"
+#include "content_craft.h"
+#include "content_nodemeta.h"
+#include "mapblock.h"
+#include "serverobject.h"
 
 #define BLOCK_EMERGE_FLAG_FROMDISK (1<<0)
 
+class MapEditEventIgnorer
+{
+public:
+       MapEditEventIgnorer(bool *flag):
+               m_flag(flag)
+       {
+               if(*m_flag == false)
+                       *m_flag = true;
+               else
+                       m_flag = NULL;
+       }
+
+       ~MapEditEventIgnorer()
+       {
+               if(m_flag)
+               {
+                       assert(*m_flag);
+                       *m_flag = false;
+               }
+       }
+       
+private:
+       bool *m_flag;
+};
+
 void * ServerThread::Thread()
 {
        ThreadStarted();
@@ -75,10 +105,10 @@ void * EmergeThread::Thread()
 
        DSTACK(__FUNCTION_NAME);
 
-       //bool debug=false;
-       
        BEGIN_DEBUG_EXCEPTION_HANDLER
 
+       bool enable_mapgen_debug_info = g_settings.getBool("enable_mapgen_debug_info");
+       
        /*
                Get block info from queue, emerge them and send them
                to clients.
@@ -125,7 +155,7 @@ void * EmergeThread::Thread()
                        Also decrement the emerge queue count in clients.
                */
 
-               bool optional = true;
+               bool only_from_disk = true;
 
                {
                        core::map<u16, u8>::Iterator i;
@@ -136,55 +166,25 @@ void * EmergeThread::Thread()
                                // Check flags
                                u8 flags = i.getNode()->getValue();
                                if((flags & BLOCK_EMERGE_FLAG_FROMDISK) == false)
-                                       optional = false;
+                                       only_from_disk = false;
                                
                        }
                }
-
-               /*dstream<<"EmergeThread: p="
-                               <<"("<<p.X<<","<<p.Y<<","<<p.Z<<") "
-                               <<"optional="<<optional<<std::endl;*/
+               
+               if(enable_mapgen_debug_info)
+                       dstream<<"EmergeThread: p="
+                                       <<"("<<p.X<<","<<p.Y<<","<<p.Z<<") "
+                                       <<"only_from_disk="<<only_from_disk<<std::endl;
                
                ServerMap &map = ((ServerMap&)m_server->m_env.getMap());
                        
-               core::map<v3s16, MapBlock*> changed_blocks;
-               core::map<v3s16, MapBlock*> lighting_invalidated_blocks;
+               //core::map<v3s16, MapBlock*> changed_blocks;
+               //core::map<v3s16, MapBlock*> lighting_invalidated_blocks;
 
                MapBlock *block = NULL;
                bool got_block = true;
                core::map<v3s16, MapBlock*> modified_blocks;
                
-               bool only_from_disk = false;
-               
-               if(optional)
-                       only_from_disk = true;
-
-               v2s16 chunkpos = map.sector_to_chunk(p2d);
-
-               bool generate_chunk = false;
-               if(only_from_disk == false)
-               {
-                       JMutexAutoLock envlock(m_server->m_env_mutex);
-                       if(map.chunkNonVolatile(chunkpos) == false)
-                               generate_chunk = true;
-               }
-               if(generate_chunk)
-               {
-                       ChunkMakeData data;
-                       
-                       {
-                               JMutexAutoLock envlock(m_server->m_env_mutex);
-                               map.initChunkMake(data, chunkpos);
-                       }
-
-                       makeChunk(&data);
-
-                       {
-                               JMutexAutoLock envlock(m_server->m_env_mutex);
-                               map.finishChunkMake(data, changed_blocks);
-                       }
-               }
-       
                /*
                        Fetch block from map or generate a single block
                */
@@ -193,36 +193,66 @@ void * EmergeThread::Thread()
                        
                        // Load sector if it isn't loaded
                        if(map.getSectorNoGenerateNoEx(p2d) == NULL)
-                               map.loadSectorFull(p2d);
+                               //map.loadSectorFull(p2d);
+                               map.loadSectorMeta(p2d);
 
                        block = map.getBlockNoCreateNoEx(p);
-                       if(!block || block->isDummy())
+                       if(!block || block->isDummy() || !block->isGenerated())
                        {
-                               if(only_from_disk)
+                               if(enable_mapgen_debug_info)
+                                       dstream<<"EmergeThread: not in memory, loading"<<std::endl;
+
+                               // Get, load or create sector
+                               /*ServerMapSector *sector =
+                                               (ServerMapSector*)map.createSector(p2d);*/
+
+                               // Load/generate block
+
+                               /*block = map.emergeBlock(p, sector, changed_blocks,
+                                               lighting_invalidated_blocks);*/
+
+                               block = map.loadBlock(p);
+                               
+                               if(only_from_disk == false)
+                               {
+                                       if(block == NULL || block->isGenerated() == false)
+                                       {
+                                               if(enable_mapgen_debug_info)
+                                                       dstream<<"EmergeThread: generating"<<std::endl;
+                                               block = map.generateBlock(p, modified_blocks);
+                                       }
+                               }
+
+                               if(enable_mapgen_debug_info)
+                                       dstream<<"EmergeThread: ended up with: "
+                                                       <<analyze_block(block)<<std::endl;
+
+                               if(block == NULL)
                                {
                                        got_block = false;
                                }
                                else
                                {
-                                       // Get, load or create sector
-                                       ServerMapSector *sector =
-                                                       (ServerMapSector*)map.createSector(p2d);
-                                       // Generate block
-                                       block = map.generateBlock(p, block, sector, changed_blocks,
-                                                       lighting_invalidated_blocks);
-                                       if(block == NULL)
-                                               got_block = false;
+                                       /*
+                                               Ignore map edit events, they will not need to be
+                                               sent to anybody because the block hasn't been sent
+                                               to anybody
+                                       */
+                                       MapEditEventIgnorer ign(&m_server->m_ignore_map_edit_events);
+                                       
+                                       // Activate objects and stuff
+                                       m_server->m_env.activateBlock(block, 3600);
                                }
                        }
                        else
                        {
-                               if(block->getLightingExpired()){
+                               /*if(block->getLightingExpired()){
                                        lighting_invalidated_blocks[block->getPos()] = block;
-                               }
+                               }*/
                        }
 
                        // TODO: Some additional checking and lighting updating,
-                       // see emergeBlock
+                       //       see emergeBlock
                }
 
                {//envlock
@@ -234,7 +264,8 @@ void * EmergeThread::Thread()
                                Collect a list of blocks that have been modified in
                                addition to the fetched one.
                        */
-                       
+
+#if 0
                        if(lighting_invalidated_blocks.size() > 0)
                        {
                                /*dstream<<"lighting "<<lighting_invalidated_blocks.size()
@@ -255,11 +286,12 @@ void * EmergeThread::Thread()
                                MapBlock *block = i.getNode()->getValue();
                                modified_blocks.insert(block->getPos(), block);
                        }
+#endif
                }
                // If we got no block, there should be no invalidated blocks
                else
                {
-                       assert(lighting_invalidated_blocks.size() == 0);
+                       //assert(lighting_invalidated_blocks.size() == 0);
                }
 
                }//envlock
@@ -348,8 +380,7 @@ void RemoteClient::GetNextBlocks(Server *server, float dtime,
        v3s16 center = getNodeBlockPos(center_nodepos);
        
        // Camera position and direction
-       v3f camera_pos =
-                       playerpos + v3f(0, BS+BS/2, 0);
+       v3f camera_pos = player->getEyePosition();
        v3f camera_dir = v3f(0,0,1);
        camera_dir.rotateYZBy(player->getPitch());
        camera_dir.rotateXZBy(player->getYaw());
@@ -553,7 +584,7 @@ void RemoteClient::GetNextBlocks(Server *server, float dtime,
                                Don't generate or send if not in sight
                        */
 
-                       if(isBlockInSight(p, camera_pos, camera_dir, 10000*BS) == false)
+                       if(isBlockInSight(p, camera_pos, camera_dir, M_PI, 10000*BS) == false)
                        {
                                continue;
                        }
@@ -577,6 +608,9 @@ void RemoteClient::GetNextBlocks(Server *server, float dtime,
                        bool block_is_invalid = false;
                        if(block != NULL)
                        {
+                               // Reset usage timer, this block will be of use in the future.
+                               block->resetUsageTimer();
+
                                // Block is dummy if data doesn't exist.
                                // It means it has been not found from disk and not generated
                                if(block->isDummy())
@@ -594,12 +628,16 @@ void RemoteClient::GetNextBlocks(Server *server, float dtime,
                                {
                                        block_is_invalid = true;
                                }*/
-                               
+
+#if 0
                                v2s16 p2d(p.X, p.Z);
                                ServerMap *map = (ServerMap*)(&server->m_env.getMap());
                                v2s16 chunkpos = map->sector_to_chunk(p2d);
                                if(map->chunkNonVolatile(chunkpos) == false)
                                        block_is_invalid = true;
+#endif
+                               if(block->isGenerated() == false)
+                                       block_is_invalid = true;
 #if 1
                                /*
                                        If block is not close, don't send it unless it is near
@@ -646,6 +684,7 @@ void RemoteClient::GetNextBlocks(Server *server, float dtime,
                                //TODO: Get value from somewhere
                                // Allow only one block in emerge queue
                                //if(server->m_emerge_queue.peerItemCount(peer_id) < 1)
+                               // Allow two blocks in queue per client
                                if(server->m_emerge_queue.peerItemCount(peer_id) < 2)
                                {
                                        //dstream<<"Adding block to emerge queue"<<std::endl;
@@ -735,6 +774,7 @@ void RemoteClient::SendObjectData(
                u16 command
                u16 number of player positions
                for each player:
+                       u16 peer_id
                        v3s32 position*100
                        v3s32 speed*100
                        s32 pitch*100
@@ -870,7 +910,9 @@ void RemoteClient::SendObjectData(
                        bos.write((char*)buf, 6);
 
                        // Write objects
-                       block->serializeObjects(bos, serialization_version);
+                       //block->serializeObjects(bos, serialization_version); // DEPRECATED
+                       // count=0
+                       writeU16(bos, 0);
 
                        blockcount++;
 
@@ -1016,17 +1058,20 @@ u32 PIChecksum(core::list<PlayerInfo> &l)
 */
 
 Server::Server(
-               std::string mapsavedir
+               std::string mapsavedir,
+               std::string configpath
        ):
        m_env(new ServerMap(mapsavedir), this),
        m_con(PROTOCOL_ID, 512, CONNECTION_TIMEOUT, this),
        m_authmanager(mapsavedir+"/auth.txt"),
+       m_banmanager(mapsavedir+"/ipban.txt"),
        m_thread(this),
        m_emergethread(this),
        m_time_counter(0),
        m_time_of_day_send_timer(0),
        m_uptime(0),
        m_mapsavedir(mapsavedir),
+       m_configpath(configpath),
        m_shutdown_requested(false),
        m_ignore_map_edit_events(false),
        m_ignore_map_edit_events_peer_id(0)
@@ -1208,6 +1253,20 @@ void Server::AsyncRunStep()
                m_uptime.set(m_uptime.get() + dtime);
        }
        
+       {
+               // Process connection's timeouts
+               JMutexAutoLock lock2(m_con_mutex);
+               ScopeProfiler sp(&g_profiler, "Server: connection timeout processing");
+               m_con.RunTimeouts(dtime);
+       }
+       
+       {
+               // This has to be called so that the client list gets synced
+               // with the peer list of the connection
+               ScopeProfiler sp(&g_profiler, "Server: peer change handling");
+               handlePeerChanges();
+       }
+
        /*
                Update m_time_of_day and overall game time
        */
@@ -1251,26 +1310,21 @@ void Server::AsyncRunStep()
        }
 
        {
-               // Process connection's timeouts
-               JMutexAutoLock lock2(m_con_mutex);
-               ScopeProfiler sp(&g_profiler, "Server: connection timeout processing");
-               m_con.RunTimeouts(dtime);
-       }
-       
-       {
-               // This has to be called so that the client list gets synced
-               // with the peer list of the connection
-               ScopeProfiler sp(&g_profiler, "Server: peer change handling");
-               handlePeerChanges();
-       }
-
-       {
-               // Step environment
-               // This also runs Map's timers
                JMutexAutoLock lock(m_env_mutex);
+               // Step environment
                ScopeProfiler sp(&g_profiler, "Server: environment step");
                m_env.step(dtime);
        }
+               
+       const float map_timer_and_unload_dtime = 5.15;
+       if(m_map_timer_and_unload_interval.step(dtime, map_timer_and_unload_dtime))
+       {
+               JMutexAutoLock lock(m_env_mutex);
+               // Run Map's timers and unload unused data
+               ScopeProfiler sp(&g_profiler, "Server: map timer and unload");
+               m_env.getMap().timerUpdate(map_timer_and_unload_dtime,
+                               g_settings.getFloat("server_unload_unused_data_timeout"));
+       }
        
        /*
                Do background stuff
@@ -1625,38 +1679,119 @@ void Server::AsyncRunStep()
                Send queued-for-sending map edit events.
        */
        {
+               // Don't send too many at a time
+               //u32 count = 0;
+
+               // Single change sending is disabled if queue size is not small
+               bool disable_single_change_sending = false;
+               if(m_unsent_map_edit_queue.size() >= 4)
+                       disable_single_change_sending = true;
+
+               bool got_any_events = false;
+
+               // We'll log the amount of each
+               Profiler prof;
+
                while(m_unsent_map_edit_queue.size() != 0)
                {
+                       got_any_events = true;
+
                        MapEditEvent* event = m_unsent_map_edit_queue.pop_front();
+                       
+                       // Players far away from the change are stored here.
+                       // Instead of sending the changes, MapBlocks are set not sent
+                       // for them.
+                       core::list<u16> far_players;
 
                        if(event->type == MEET_ADDNODE)
                        {
-                               dstream<<"Server: MEET_ADDNODE"<<std::endl;
-                               sendAddNode(event->p, event->n, event->already_known_by_peer);
+                               //dstream<<"Server: MEET_ADDNODE"<<std::endl;
+                               prof.add("MEET_ADDNODE", 1);
+                               if(disable_single_change_sending)
+                                       sendAddNode(event->p, event->n, event->already_known_by_peer,
+                                                       &far_players, 5);
+                               else
+                                       sendAddNode(event->p, event->n, event->already_known_by_peer,
+                                                       &far_players, 30);
                        }
                        else if(event->type == MEET_REMOVENODE)
                        {
-                               dstream<<"Server: MEET_REMOVENODE"<<std::endl;
-                               sendRemoveNode(event->p, event->already_known_by_peer);
+                               //dstream<<"Server: MEET_REMOVENODE"<<std::endl;
+                               prof.add("MEET_REMOVENODE", 1);
+                               if(disable_single_change_sending)
+                                       sendRemoveNode(event->p, event->already_known_by_peer,
+                                                       &far_players, 5);
+                               else
+                                       sendRemoveNode(event->p, event->already_known_by_peer,
+                                                       &far_players, 30);
                        }
                        else if(event->type == MEET_BLOCK_NODE_METADATA_CHANGED)
                        {
                                dstream<<"Server: MEET_BLOCK_NODE_METADATA_CHANGED"<<std::endl;
+                               prof.add("MEET_BLOCK_NODE_METADATA_CHANGED", 1);
                                setBlockNotSent(event->p);
                        }
                        else if(event->type == MEET_OTHER)
                        {
-                               dstream<<"WARNING: Server: MEET_OTHER not implemented"
-                                               <<std::endl;
+                               dstream<<"Server: MEET_OTHER"<<std::endl;
+                               prof.add("MEET_OTHER", 1);
+                               for(core::map<v3s16, bool>::Iterator
+                                               i = event->modified_blocks.getIterator();
+                                               i.atEnd()==false; i++)
+                               {
+                                       v3s16 p = i.getNode()->getKey();
+                                       setBlockNotSent(p);
+                               }
                        }
                        else
                        {
+                               prof.add("unknown", 1);
                                dstream<<"WARNING: Server: Unknown MapEditEvent "
                                                <<((u32)event->type)<<std::endl;
                        }
+                       
+                       /*
+                               Set blocks not sent to far players
+                       */
+                       if(far_players.size() > 0)
+                       {
+                               // Convert list format to that wanted by SetBlocksNotSent
+                               core::map<v3s16, MapBlock*> modified_blocks2;
+                               for(core::map<v3s16, bool>::Iterator
+                                               i = event->modified_blocks.getIterator();
+                                               i.atEnd()==false; i++)
+                               {
+                                       v3s16 p = i.getNode()->getKey();
+                                       modified_blocks2.insert(p,
+                                                       m_env.getMap().getBlockNoCreateNoEx(p));
+                               }
+                               // Set blocks not sent
+                               for(core::list<u16>::Iterator
+                                               i = far_players.begin();
+                                               i != far_players.end(); i++)
+                               {
+                                       u16 peer_id = *i;
+                                       RemoteClient *client = getClient(peer_id);
+                                       if(client==NULL)
+                                               continue;
+                                       client->SetBlocksNotSent(modified_blocks2);
+                               }
+                       }
 
                        delete event;
+
+                       /*// Don't send too many at a time
+                       count++;
+                       if(count >= 1 && m_unsent_map_edit_queue.size() < 100)
+                               break;*/
                }
+
+               if(got_any_events)
+               {
+                       dstream<<"Server: MapEditEvents:"<<std::endl;
+                       prof.print(dstream);
+               }
+               
        }
 
        /*
@@ -1679,39 +1814,6 @@ void Server::AsyncRunStep()
                }
        }
        
-       /*
-               Step node metadata
-               TODO: Move to ServerEnvironment and utilize active block stuff
-       */
-       /*{
-               //TimeTaker timer("Step node metadata");
-
-               JMutexAutoLock envlock(m_env_mutex);
-               JMutexAutoLock conlock(m_con_mutex);
-
-               ScopeProfiler sp(&g_profiler, "Server: stepping node metadata");
-
-               core::map<v3s16, MapBlock*> changed_blocks;
-               m_env.getMap().nodeMetadataStep(dtime, changed_blocks);
-               
-               // Use setBlockNotSent
-
-               for(core::map<v3s16, MapBlock*>::Iterator
-                               i = changed_blocks.getIterator();
-                               i.atEnd() == false; i++)
-               {
-                       MapBlock *block = i.getNode()->getValue();
-
-                       for(core::map<u16, RemoteClient*>::Iterator
-                               i = m_clients.getIterator();
-                               i.atEnd()==false; i++)
-                       {
-                               RemoteClient *client = i.getNode()->getValue();
-                               client->SetBlockNotSent(block->getPos());
-                       }
-               }
-       }*/
-               
        /*
                Trigger emergethread (it somehow gets to a non-triggered but
                bysy state sometimes)
@@ -1740,29 +1842,36 @@ void Server::AsyncRunStep()
                        // Auth stuff
                        if(m_authmanager.isModified())
                                m_authmanager.save();
+
+                       //Bann stuff
+                       if(m_banmanager.isModified())
+                               m_banmanager.save();
                        
                        // Map
                        JMutexAutoLock lock(m_env_mutex);
-                       if(((ServerMap*)(&m_env.getMap()))->isSavingEnabled() == true)
-                       {
-                               // Save only changed parts
-                               m_env.getMap().save(true);
 
-                               // Delete unused sectors
-                               u32 deleted_count = m_env.getMap().deleteUnusedSectors(
-                                               g_settings.getFloat("server_unload_unused_sectors_timeout"));
-                               if(deleted_count > 0)
-                               {
-                                       dout_server<<"Server: Unloaded "<<deleted_count
-                                                       <<" sectors from memory"<<std::endl;
-                               }
+                       /*// Unload unused data (delete from memory)
+                       m_env.getMap().unloadUnusedData(
+                                       g_settings.getFloat("server_unload_unused_sectors_timeout"));
+                                       */
+                       /*u32 deleted_count = m_env.getMap().unloadUnusedData(
+                                       g_settings.getFloat("server_unload_unused_sectors_timeout"));
+                                       */
 
-                               // Save players
-                               m_env.serializePlayers(m_mapsavedir);
-                               
-                               // Save environment metadata
-                               m_env.saveMeta(m_mapsavedir);
-                       }
+                       // Save only changed parts
+                       m_env.getMap().save(true);
+
+                       /*if(deleted_count > 0)
+                       {
+                               dout_server<<"Server: Unloaded "<<deleted_count
+                                               <<" blocks from memory"<<std::endl;
+                       }*/
+
+                       // Save players
+                       m_env.serializePlayers(m_mapsavedir);
+                       
+                       // Save environment metadata
+                       m_env.saveMeta(m_mapsavedir);
                }
        }
 }
@@ -1826,6 +1935,16 @@ void Server::ProcessData(u8 *data, u32 datasize, u16 peer_id)
                                <<peer_id<<" not found"<<std::endl;
                return;
        }
+
+       // drop player if is ip is banned
+       if(m_banmanager.isIpBanned(peer->address.serializeString())){
+               SendAccessDenied(m_con, peer_id,
+                               L"Your ip is banned. Banned name was "
+                               +narrow_to_wide(m_banmanager.getBanName(
+                                       peer->address.serializeString())));
+               m_con.deletePeer(peer_id, false);
+               return;
+       }
        
        u8 peer_ser_ver = getClient(peer->id)->serialization_version;
 
@@ -1862,12 +1981,33 @@ void Server::ProcessData(u8 *data, u32 datasize, u16 peer_id)
 
                //peer->serialization_version = deployed;
                getClient(peer->id)->pending_serialization_version = deployed;
-
+               
                if(deployed == SER_FMT_VER_INVALID)
                {
                        derr_server<<DTIME<<"Server: Cannot negotiate "
                                        "serialization version with peer "
                                        <<peer_id<<std::endl;
+                       SendAccessDenied(m_con, peer_id,
+                                       L"Your client is too old (map format)");
+                       return;
+               }
+               
+               /*
+                       Read and check network protocol version
+               */
+
+               u16 net_proto_version = 0;
+               if(datasize >= 2+1+PLAYERNAME_SIZE+PASSWORD_SIZE+2)
+               {
+                       net_proto_version = readU16(&data[2+1+PLAYERNAME_SIZE+PASSWORD_SIZE]);
+               }
+
+               getClient(peer->id)->net_proto_version = net_proto_version;
+
+               if(net_proto_version == 0)
+               {
+                       SendAccessDenied(m_con, peer_id,
+                                       L"Your client is too old. Please upgrade.");
                        return;
                }
 
@@ -1901,7 +2041,7 @@ void Server::ProcessData(u8 *data, u32 datasize, u16 peer_id)
 
                // Get password
                char password[PASSWORD_SIZE];
-               if(datasize == 2+1+PLAYERNAME_SIZE)
+               if(datasize < 2+1+PLAYERNAME_SIZE+PASSWORD_SIZE)
                {
                        // old version - assume blank password
                        password[0] = 0;
@@ -1925,7 +2065,10 @@ void Server::ProcessData(u8 *data, u32 datasize, u16 peer_id)
                        checkpwd = g_settings.get("default_password");
                }
                
-               if(password != checkpwd && checkpwd != "")
+               /*dstream<<"Server: Client gave password '"<<password
+                               <<"', the correct one is '"<<checkpwd<<"'"<<std::endl;*/
+               
+               if(password != checkpwd && m_authmanager.exists(playername))
                {
                        derr_server<<DTIME<<"Server: peer_id="<<peer_id
                                        <<": supplied invalid password for "
@@ -2036,6 +2179,9 @@ void Server::ProcessData(u8 *data, u32 datasize, u16 peer_id)
                UpdateCrafting(peer->id);
                SendInventory(peer->id);
 
+               // Send player items to all players
+               SendPlayerItems();
+
                // Send HP
                {
                        Player *player = m_env.getPlayer(peer_id);
@@ -2065,6 +2211,12 @@ void Server::ProcessData(u8 *data, u32 datasize, u16 peer_id)
                        message += L" joined game";
                        BroadcastChatMessage(message);
                }
+               
+               // Warnings about protocol version can be issued here
+               /*if(getClient(peer->id)->net_proto_version == 0)
+               {
+                       SendChatMessage(peer_id, L"# Server: NOTE: YOUR CLIENT IS OLD AND DOES NOT WORK PROPERLY WITH THIS SERVER");
+               }*/
 
                return;
        }
@@ -2272,71 +2424,92 @@ void Server::ProcessData(u8 *data, u32 datasize, u16 peer_id)
                        return;
                }
 
+               // Skip if object has been removed
+               if(obj->m_removed)
+                       return;
+               
                //TODO: Check that object is reasonably close
                
                // Left click, pick object up (usually)
                if(button == 0)
                {
-                       InventoryList *ilist = player->inventory.getList("main");
-                       if(g_settings.getBool("creative_mode") == false && ilist != NULL)
-                       {
+                       /*
+                               Try creating inventory item
+                       */
+                       InventoryItem *item = obj->createPickedUpItem();
                        
-                               // Skip if inventory has no free space
-                               if(ilist->getUsedSlots() == ilist->getSize())
+                       if(item)
+                       {
+                               InventoryList *ilist = player->inventory.getList("main");
+                               if(ilist != NULL)
                                {
-                                       dout_server<<"Player inventory has no free space"<<std::endl;
-                                       return;
-                               }
+                                       if(g_settings.getBool("creative_mode") == false)
+                                       {
+                                               // Skip if inventory has no free space
+                                               if(ilist->roomForItem(item) == false)
+                                               {
+                                                       dout_server<<"Player inventory has no free space"<<std::endl;
+                                                       return;
+                                               }
 
-                               // Skip if object has been removed
-                               if(obj->m_removed)
-                                       return;
-                               
-                               /*
-                                       Create the inventory item
-                               */
-                               InventoryItem *item = obj->createPickedUpItem();
-                               
-                               if(item)
-                               {
-                                       // Add to inventory and send inventory
-                                       ilist->addItem(item);
-                                       UpdateCrafting(player->peer_id);
-                                       SendInventory(player->peer_id);
+                                               // Add to inventory and send inventory
+                                               ilist->addItem(item);
+                                               UpdateCrafting(player->peer_id);
+                                               SendInventory(player->peer_id);
+                                       }
 
                                        // Remove object from environment
                                        obj->m_removed = true;
                                }
-                               else
-                               {
-                                       /*
-                                               Item cannot be picked up. Punch it instead.
-                                       */
+                       }
+                       else
+                       {
+                               /*
+                                       Item cannot be picked up. Punch it instead.
+                               */
 
-                                       ToolItem *titem = NULL;
-                                       std::string toolname = "";
+                               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();
-                                               }
-                                       }
-                                       
-                                       u16 wear = obj->punch(toolname);
-                                       
-                                       if(titem)
+                               InventoryList *mlist = player->inventory.getList("main");
+                               if(mlist != NULL)
+                               {
+                                       InventoryItem *item = mlist->getItem(item_i);
+                                       if(item && (std::string)item->getName() == "ToolItem")
                                        {
-                                               bool weared_out = titem->addWear(wear);
-                                               if(weared_out)
-                                                       mlist->deleteItem(item_i);
-                                               SendInventory(player->peer_id);
+                                               titem = (ToolItem*)item;
+                                               toolname = titem->getToolName();
                                        }
                                }
+
+                               v3f playerpos = player->getPosition();
+                               v3f objpos = obj->getBasePosition();
+                               v3f dir = (objpos - playerpos).normalize();
+                               
+                               u16 wear = obj->punch(toolname, dir);
+                               
+                               if(titem)
+                               {
+                                       bool weared_out = titem->addWear(wear);
+                                       if(weared_out)
+                                               mlist->deleteItem(item_i);
+                                       SendInventory(player->peer_id);
+                               }
+                       }
+               }
+               // Right click, do something with object
+               if(button == 1)
+               {
+                       // Track hp changes super-crappily
+                       u16 oldhp = player->hp;
+                       
+                       // Do stuff
+                       obj->rightClick(player);
+                       
+                       // Send back stuff
+                       if(player->hp != oldhp)
+                       {
+                               SendPlayerHP(player);
                        }
                }
        }
@@ -2401,7 +2574,7 @@ void Server::ProcessData(u8 *data, u32 datasize, u16 peer_id)
                        // Mandatory parameter; actually used for nothing
                        core::map<v3s16, MapBlock*> modified_blocks;
 
-                       u8 material = CONTENT_IGNORE;
+                       content_t material = CONTENT_IGNORE;
                        u8 mineral = MINERAL_NONE;
 
                        bool cannot_remove_node = false;
@@ -2412,7 +2585,7 @@ void Server::ProcessData(u8 *data, u32 datasize, u16 peer_id)
                                // Get mineral
                                mineral = n.getMineral();
                                // Get material at position
-                               material = n.d;
+                               material = n.getContent();
                                // If not yet cancelled
                                if(cannot_remove_node == false)
                                {
@@ -2480,12 +2653,12 @@ void Server::ProcessData(u8 *data, u32 datasize, u16 peer_id)
                        }
                        
                        /*
-                               Send the removal to all other clients.
+                               Send the removal to all close-by players.
                                - If other player is close, send REMOVENODE
                                - Otherwise set blocks not sent
                        */
                        core::list<u16> far_players;
-                       sendRemoveNode(p_under, peer_id, &far_players, 100);
+                       sendRemoveNode(p_under, peer_id, &far_players, 30);
                        
                        /*
                                Update and send inventory
@@ -2560,10 +2733,11 @@ void Server::ProcessData(u8 *data, u32 datasize, u16 peer_id)
                                Remove the node
                                (this takes some time so it is done after the quick stuff)
                        */
-                       m_ignore_map_edit_events = true;
-                       m_env.getMap().removeNodeAndUpdate(p_under, modified_blocks);
-                       m_ignore_map_edit_events = false;
-                       
+                       {
+                               MapEditEventIgnorer ign(&m_ignore_map_edit_events);
+
+                               m_env.getMap().removeNodeAndUpdate(p_under, modified_blocks);
+                       }
                        /*
                                Set blocks not sent to far players
                        */
@@ -2611,7 +2785,7 @@ void Server::ProcessData(u8 *data, u32 datasize, u16 peer_id)
                                                        <<" because privileges are "<<getPlayerPrivs(player)
                                                        <<std::endl;
 
-                                       if(content_buildable_to(n2.d) == false
+                                       if(content_features(n2).buildable_to == false
                                                || no_enough_privs)
                                        {
                                                // Client probably has wrong data.
@@ -2642,15 +2816,37 @@ void Server::ProcessData(u8 *data, u32 datasize, u16 peer_id)
                                // Create node data
                                MaterialItem *mitem = (MaterialItem*)item;
                                MapNode n;
-                               n.d = mitem->getMaterial();
-                               if(content_features(n.d).wall_mounted)
-                                       n.dir = packDir(p_under - p_over);
-                               
+                               n.setContent(mitem->getMaterial());
+
+                               // Calculate direction for wall mounted stuff
+                               if(content_features(n).wall_mounted)
+                                       n.param2 = packDir(p_under - p_over);
+
+                               // Calculate the direction for furnaces and chests and stuff
+                               if(content_features(n).param_type == CPT_FACEDIR_SIMPLE)
+                               {
+                                       v3f playerpos = player->getPosition();
+                                       v3f blockpos = intToFloat(p_over, BS) - playerpos;
+                                       blockpos = blockpos.normalize();
+                                       n.param1 = 0;
+                                       if (fabs(blockpos.X) > fabs(blockpos.Z)) {
+                                               if (blockpos.X < 0)
+                                                       n.param1 = 3;
+                                               else
+                                                       n.param1 = 1;
+                                       } else {
+                                               if (blockpos.Z < 0)
+                                                       n.param1 = 2;
+                                               else
+                                                       n.param1 = 0;
+                                       }
+                               }
+
                                /*
-                                       Send to all players
+                                       Send to all close-by players
                                */
                                core::list<u16> far_players;
-                               sendAddNode(p_over, n, 0, &far_players, 100);
+                               sendAddNode(p_over, n, 0, &far_players, 30);
                                
                                /*
                                        Handle inventory
@@ -2674,10 +2870,11 @@ void Server::ProcessData(u8 *data, u32 datasize, u16 peer_id)
                                        This takes some time so it is done after the quick stuff
                                */
                                core::map<v3s16, MapBlock*> modified_blocks;
-                               m_ignore_map_edit_events = true;
-                               m_env.getMap().addNodeAndUpdate(p_over, n, modified_blocks);
-                               m_ignore_map_edit_events = false;
-                               
+                               {
+                                       MapEditEventIgnorer ign(&m_ignore_map_edit_events);
+
+                                       m_env.getMap().addNodeAndUpdate(p_over, n, modified_blocks);
+                               }
                                /*
                                        Set blocks not sent to far players
                                */
@@ -2726,6 +2923,18 @@ void Server::ProcessData(u8 *data, u32 datasize, u16 peer_id)
                                        return;
                                }
 
+                               /*
+                                       If in creative mode, item dropping is disabled unless
+                                       player has build privileges
+                               */
+                               if(g_settings.getBool("creative_mode") &&
+                                       (getPlayerPrivs(player) & PRIV_BUILD) == 0)
+                               {
+                                       derr_server<<"Not allowing player to drop item: "
+                                                       "creative mode and no build privs"<<std::endl;
+                                       return;
+                               }
+
                                dout_server<<"Placing a miscellaneous item on map"
                                                <<std::endl;
                                
@@ -2989,6 +3198,9 @@ void Server::ProcessData(u8 *data, u32 datasize, u16 peer_id)
                                                mlist->addItem(item1);
                                        }
                                }
+                               // Disallow moving items if not allowed to build
+                               else if((getPlayerPrivs(player) & PRIV_BUILD) == 0)
+                                       return;
                        }
                        
                        if(disable_action == false)
@@ -3049,23 +3261,34 @@ void Server::ProcessData(u8 *data, u32 datasize, u16 peer_id)
                u64 privs = getPlayerPrivs(player);
 
                // Parse commands
-               std::wstring commandprefix = L"/#";
-               if(message.substr(0, commandprefix.size()) == commandprefix)
+               if(message[0] == L'/')
                {
-                       line += L"Server: ";
+                       size_t strip_size = 1;
+                       if (message[1] == L'#') // support old-style commans
+                               ++strip_size;
+                       message = message.substr(strip_size);
 
-                       message = message.substr(commandprefix.size());
+                       WStrfnd f1(message);
+                       f1.next(L" "); // Skip over /#whatever
+                       std::wstring paramstring = f1.next(L"");
 
                        ServerCommandContext *ctx = new ServerCommandContext(
                                str_split(message, L' '),
+                               paramstring,
                                this,
                                &m_env,
                                player,
                                privs);
 
-                       line += processServerCommand(ctx);
-                       send_to_sender = ctx->flags & 1;
-                       send_to_others = ctx->flags & 2;
+                       std::wstring reply(processServerCommand(ctx));
+                       send_to_sender = ctx->flags & SEND_TO_SENDER;
+                       send_to_others = ctx->flags & SEND_TO_OTHERS;
+
+                       if (ctx->flags & SEND_NO_PREFIX)
+                               line += reply;
+                       else
+                               line += L"Server: " + reply;
+
                        delete ctx;
 
                }
@@ -3175,6 +3398,9 @@ void Server::ProcessData(u8 *data, u32 datasize, u16 peer_id)
                        newpwd += c;
                }
 
+               dstream<<"Server: Client requests a password change from "
+                               <<"'"<<oldpwd<<"' to '"<<newpwd<<"'"<<std::endl;
+
                std::string playername = player->getName();
 
                if(m_authmanager.exists(playername) == false)
@@ -3186,7 +3412,7 @@ void Server::ProcessData(u8 *data, u32 datasize, u16 peer_id)
                }
 
                std::string checkpwd = m_authmanager.getPassword(playername);
-               
+
                if(oldpwd != checkpwd)
                {
                        dstream<<"Server: invalid old password"<<std::endl;
@@ -3201,6 +3427,15 @@ void Server::ProcessData(u8 *data, u32 datasize, u16 peer_id)
                                <<std::endl;
                SendChatMessage(peer_id, L"Password change successful");
        }
+       else if (command == TOSERVER_PLAYERITEM)
+       {
+               if (datasize < 2+2)
+                       return;
+
+               u16 item = readU16(&data[2]);
+               player->wieldItem(item);
+               SendWieldedItem(player);
+       }
        else
        {
                derr_server<<"WARNING: Server::ProcessData(): Ignoring "
@@ -3218,7 +3453,7 @@ void Server::ProcessData(u8 *data, u32 datasize, u16 peer_id)
 
 void Server::onMapEditEvent(MapEditEvent *event)
 {
-       dstream<<"Server::onMapEditEvent()"<<std::endl;
+       //dstream<<"Server::onMapEditEvent()"<<std::endl;
        if(m_ignore_map_edit_events)
                return;
        MapEditEvent *e = event->clone();
@@ -3485,6 +3720,60 @@ void Server::SendInventory(u16 peer_id)
        m_con.Send(peer_id, 0, data, true);
 }
 
+std::string getWieldedItemString(const Player *player)
+{
+       const InventoryItem *item = player->getWieldItem();
+       if (item == NULL)
+               return std::string("");
+       std::ostringstream os(std::ios_base::binary);
+       item->serialize(os);
+       return os.str();
+}
+
+void Server::SendWieldedItem(const Player* player)
+{
+       DSTACK(__FUNCTION_NAME);
+
+       assert(player);
+
+       std::ostringstream os(std::ios_base::binary);
+
+       writeU16(os, TOCLIENT_PLAYERITEM);
+       writeU16(os, 1);
+       writeU16(os, player->peer_id);
+       os<<serializeString(getWieldedItemString(player));
+
+       // Make data buffer
+       std::string s = os.str();
+       SharedBuffer<u8> data((u8*)s.c_str(), s.size());
+
+       m_con.SendToAll(0, data, true);
+}
+
+void Server::SendPlayerItems()
+{
+       DSTACK(__FUNCTION_NAME);
+
+       std::ostringstream os(std::ios_base::binary);
+       core::list<Player *> players = m_env.getPlayers(true);
+
+       writeU16(os, TOCLIENT_PLAYERITEM);
+       writeU16(os, players.size());
+       core::list<Player *>::Iterator i;
+       for(i = players.begin(); i != players.end(); ++i)
+       {
+               Player *p = *i;
+               writeU16(os, p->peer_id);
+               os<<serializeString(getWieldedItemString(p));
+       }
+
+       // Make data buffer
+       std::string s = os.str();
+       SharedBuffer<u8> data((u8*)s.c_str(), s.size());
+
+       m_con.SendToAll(0, data, true);
+}
+
 void Server::SendChatMessage(u16 peer_id, const std::wstring &message)
 {
        DSTACK(__FUNCTION_NAME);
@@ -3827,334 +4116,10 @@ void Server::UpdateCrafting(u16 peer_id)
                                items[i] = clist->getItem(i);
                        }
                        
-                       bool found = false;
-
-                       // Wood
-                       if(!found)
-                       {
-                               ItemSpec specs[9];
-                               specs[0] = ItemSpec(ITEM_MATERIAL, CONTENT_TREE);
-                               if(checkItemCombination(items, specs))
-                               {
-                                       rlist->addItem(new MaterialItem(CONTENT_WOOD, 4));
-                                       found = true;
-                               }
-                       }
-
-                       // Stick
-                       if(!found)
-                       {
-                               ItemSpec specs[9];
-                               specs[0] = ItemSpec(ITEM_MATERIAL, CONTENT_WOOD);
-                               if(checkItemCombination(items, specs))
-                               {
-                                       rlist->addItem(new CraftItem("Stick", 4));
-                                       found = true;
-                               }
-                       }
-
-                       // Fence
-                       if(!found)
-                       {
-                               ItemSpec specs[9];
-                               specs[3] = ItemSpec(ITEM_CRAFT, "Stick");
-                               specs[4] = ItemSpec(ITEM_CRAFT, "Stick");
-                               specs[5] = ItemSpec(ITEM_CRAFT, "Stick");
-                               specs[6] = ItemSpec(ITEM_CRAFT, "Stick");
-                               specs[7] = ItemSpec(ITEM_CRAFT, "Stick");
-                               specs[8] = ItemSpec(ITEM_CRAFT, "Stick");
-                               if(checkItemCombination(items, specs))
-                               {
-                                       rlist->addItem(new MaterialItem(CONTENT_FENCE, 2));
-                                       found = true;
-                               }
-                       }
-
-                       // Sign
-                       if(!found)
-                       {
-                               ItemSpec specs[9];
-                               specs[0] = ItemSpec(ITEM_MATERIAL, CONTENT_WOOD);
-                               specs[1] = ItemSpec(ITEM_MATERIAL, CONTENT_WOOD);
-                               specs[2] = ItemSpec(ITEM_MATERIAL, CONTENT_WOOD);
-                               specs[3] = ItemSpec(ITEM_MATERIAL, CONTENT_WOOD);
-                               specs[4] = ItemSpec(ITEM_MATERIAL, CONTENT_WOOD);
-                               specs[5] = ItemSpec(ITEM_MATERIAL, CONTENT_WOOD);
-                               specs[7] = ItemSpec(ITEM_CRAFT, "Stick");
-                               if(checkItemCombination(items, specs))
-                               {
-                                       //rlist->addItem(new MapBlockObjectItem("Sign"));
-                                       rlist->addItem(new MaterialItem(CONTENT_SIGN_WALL, 1));
-                                       found = true;
-                               }
-                       }
-
-                       // Torch
-                       if(!found)
-                       {
-                               ItemSpec specs[9];
-                               specs[0] = ItemSpec(ITEM_CRAFT, "lump_of_coal");
-                               specs[3] = ItemSpec(ITEM_CRAFT, "Stick");
-                               if(checkItemCombination(items, specs))
-                               {
-                                       rlist->addItem(new MaterialItem(CONTENT_TORCH, 4));
-                                       found = true;
-                               }
-                       }
-
-                       // Wooden pick
-                       if(!found)
-                       {
-                               ItemSpec specs[9];
-                               specs[0] = ItemSpec(ITEM_MATERIAL, CONTENT_WOOD);
-                               specs[1] = ItemSpec(ITEM_MATERIAL, CONTENT_WOOD);
-                               specs[2] = ItemSpec(ITEM_MATERIAL, CONTENT_WOOD);
-                               specs[4] = ItemSpec(ITEM_CRAFT, "Stick");
-                               specs[7] = ItemSpec(ITEM_CRAFT, "Stick");
-                               if(checkItemCombination(items, specs))
-                               {
-                                       rlist->addItem(new ToolItem("WPick", 0));
-                                       found = true;
-                               }
-                       }
-
-                       // Stone pick
-                       if(!found)
-                       {
-                               ItemSpec specs[9];
-                               specs[0] = ItemSpec(ITEM_MATERIAL, CONTENT_COBBLE);
-                               specs[1] = ItemSpec(ITEM_MATERIAL, CONTENT_COBBLE);
-                               specs[2] = ItemSpec(ITEM_MATERIAL, CONTENT_COBBLE);
-                               specs[4] = ItemSpec(ITEM_CRAFT, "Stick");
-                               specs[7] = ItemSpec(ITEM_CRAFT, "Stick");
-                               if(checkItemCombination(items, specs))
-                               {
-                                       rlist->addItem(new ToolItem("STPick", 0));
-                                       found = true;
-                               }
-                       }
-
-                       // Steel pick
-                       if(!found)
-                       {
-                               ItemSpec specs[9];
-                               specs[0] = ItemSpec(ITEM_CRAFT, "steel_ingot");
-                               specs[1] = ItemSpec(ITEM_CRAFT, "steel_ingot");
-                               specs[2] = ItemSpec(ITEM_CRAFT, "steel_ingot");
-                               specs[4] = ItemSpec(ITEM_CRAFT, "Stick");
-                               specs[7] = ItemSpec(ITEM_CRAFT, "Stick");
-                               if(checkItemCombination(items, specs))
-                               {
-                                       rlist->addItem(new ToolItem("SteelPick", 0));
-                                       found = true;
-                               }
-                       }
-
-                       // Mese pick
-                       if(!found)
-                       {
-                               ItemSpec specs[9];
-                               specs[0] = ItemSpec(ITEM_MATERIAL, CONTENT_MESE);
-                               specs[1] = ItemSpec(ITEM_MATERIAL, CONTENT_MESE);
-                               specs[2] = ItemSpec(ITEM_MATERIAL, CONTENT_MESE);
-                               specs[4] = ItemSpec(ITEM_CRAFT, "Stick");
-                               specs[7] = ItemSpec(ITEM_CRAFT, "Stick");
-                               if(checkItemCombination(items, specs))
-                               {
-                                       rlist->addItem(new ToolItem("MesePick", 0));
-                                       found = true;
-                               }
-                       }
-
-                       // Wooden shovel
-                       if(!found)
-                       {
-                               ItemSpec specs[9];
-                               specs[1] = ItemSpec(ITEM_MATERIAL, CONTENT_WOOD);
-                               specs[4] = ItemSpec(ITEM_CRAFT, "Stick");
-                               specs[7] = ItemSpec(ITEM_CRAFT, "Stick");
-                               if(checkItemCombination(items, specs))
-                               {
-                                       rlist->addItem(new ToolItem("WShovel", 0));
-                                       found = true;
-                               }
-                       }
-
-                       // Stone shovel
-                       if(!found)
-                       {
-                               ItemSpec specs[9];
-                               specs[1] = ItemSpec(ITEM_MATERIAL, CONTENT_COBBLE);
-                               specs[4] = ItemSpec(ITEM_CRAFT, "Stick");
-                               specs[7] = ItemSpec(ITEM_CRAFT, "Stick");
-                               if(checkItemCombination(items, specs))
-                               {
-                                       rlist->addItem(new ToolItem("STShovel", 0));
-                                       found = true;
-                               }
-                       }
-
-                       // Steel shovel
-                       if(!found)
-                       {
-                               ItemSpec specs[9];
-                               specs[1] = ItemSpec(ITEM_CRAFT, "steel_ingot");
-                               specs[4] = ItemSpec(ITEM_CRAFT, "Stick");
-                               specs[7] = ItemSpec(ITEM_CRAFT, "Stick");
-                               if(checkItemCombination(items, specs))
-                               {
-                                       rlist->addItem(new ToolItem("SteelShovel", 0));
-                                       found = true;
-                               }
-                       }
-
-                       // Wooden axe
-                       if(!found)
-                       {
-                               ItemSpec specs[9];
-                               specs[0] = ItemSpec(ITEM_MATERIAL, CONTENT_WOOD);
-                               specs[1] = ItemSpec(ITEM_MATERIAL, CONTENT_WOOD);
-                               specs[3] = ItemSpec(ITEM_MATERIAL, CONTENT_WOOD);
-                               specs[4] = ItemSpec(ITEM_CRAFT, "Stick");
-                               specs[7] = ItemSpec(ITEM_CRAFT, "Stick");
-                               if(checkItemCombination(items, specs))
-                               {
-                                       rlist->addItem(new ToolItem("WAxe", 0));
-                                       found = true;
-                               }
-                       }
-
-                       // Stone axe
-                       if(!found)
-                       {
-                               ItemSpec specs[9];
-                               specs[0] = ItemSpec(ITEM_MATERIAL, CONTENT_COBBLE);
-                               specs[1] = ItemSpec(ITEM_MATERIAL, CONTENT_COBBLE);
-                               specs[3] = ItemSpec(ITEM_MATERIAL, CONTENT_COBBLE);
-                               specs[4] = ItemSpec(ITEM_CRAFT, "Stick");
-                               specs[7] = ItemSpec(ITEM_CRAFT, "Stick");
-                               if(checkItemCombination(items, specs))
-                               {
-                                       rlist->addItem(new ToolItem("STAxe", 0));
-                                       found = true;
-                               }
-                       }
-
-                       // Steel axe
-                       if(!found)
-                       {
-                               ItemSpec specs[9];
-                               specs[0] = ItemSpec(ITEM_CRAFT, "steel_ingot");
-                               specs[1] = ItemSpec(ITEM_CRAFT, "steel_ingot");
-                               specs[3] = ItemSpec(ITEM_CRAFT, "steel_ingot");
-                               specs[4] = ItemSpec(ITEM_CRAFT, "Stick");
-                               specs[7] = ItemSpec(ITEM_CRAFT, "Stick");
-                               if(checkItemCombination(items, specs))
-                               {
-                                       rlist->addItem(new ToolItem("SteelAxe", 0));
-                                       found = true;
-                               }
-                       }
-
-                       // Wooden sword
-                       if(!found)
-                       {
-                               ItemSpec specs[9];
-                               specs[1] = ItemSpec(ITEM_MATERIAL, CONTENT_WOOD);
-                               specs[4] = ItemSpec(ITEM_MATERIAL, CONTENT_WOOD);
-                               specs[7] = ItemSpec(ITEM_CRAFT, "Stick");
-                               if(checkItemCombination(items, specs))
-                               {
-                                       rlist->addItem(new ToolItem("WSword", 0));
-                                       found = true;
-                               }
-                       }
-
-                       // Stone sword
-                       if(!found)
-                       {
-                               ItemSpec specs[9];
-                               specs[1] = ItemSpec(ITEM_MATERIAL, CONTENT_COBBLE);
-                               specs[4] = ItemSpec(ITEM_MATERIAL, CONTENT_COBBLE);
-                               specs[7] = ItemSpec(ITEM_CRAFT, "Stick");
-                               if(checkItemCombination(items, specs))
-                               {
-                                       rlist->addItem(new ToolItem("STSword", 0));
-                                       found = true;
-                               }
-                       }
-
-                       // Steel sword
-                       if(!found)
-                       {
-                               ItemSpec specs[9];
-                               specs[1] = ItemSpec(ITEM_CRAFT, "steel_ingot");
-                               specs[4] = ItemSpec(ITEM_CRAFT, "steel_ingot");
-                               specs[7] = ItemSpec(ITEM_CRAFT, "Stick");
-                               if(checkItemCombination(items, specs))
-                               {
-                                       rlist->addItem(new ToolItem("SteelSword", 0));
-                                       found = true;
-                               }
-                       }
-
-                       // Chest
-                       if(!found)
-                       {
-                               ItemSpec specs[9];
-                               specs[0] = ItemSpec(ITEM_MATERIAL, CONTENT_WOOD);
-                               specs[1] = ItemSpec(ITEM_MATERIAL, CONTENT_WOOD);
-                               specs[2] = ItemSpec(ITEM_MATERIAL, CONTENT_WOOD);
-                               specs[3] = ItemSpec(ITEM_MATERIAL, CONTENT_WOOD);
-                               specs[5] = ItemSpec(ITEM_MATERIAL, CONTENT_WOOD);
-                               specs[6] = ItemSpec(ITEM_MATERIAL, CONTENT_WOOD);
-                               specs[7] = ItemSpec(ITEM_MATERIAL, CONTENT_WOOD);
-                               specs[8] = ItemSpec(ITEM_MATERIAL, CONTENT_WOOD);
-                               if(checkItemCombination(items, specs))
-                               {
-                                       rlist->addItem(new MaterialItem(CONTENT_CHEST, 1));
-                                       found = true;
-                               }
-                       }
-
-                       // Furnace
-                       if(!found)
-                       {
-                               ItemSpec specs[9];
-                               specs[0] = ItemSpec(ITEM_MATERIAL, CONTENT_COBBLE);
-                               specs[1] = ItemSpec(ITEM_MATERIAL, CONTENT_COBBLE);
-                               specs[2] = ItemSpec(ITEM_MATERIAL, CONTENT_COBBLE);
-                               specs[3] = ItemSpec(ITEM_MATERIAL, CONTENT_COBBLE);
-                               specs[5] = ItemSpec(ITEM_MATERIAL, CONTENT_COBBLE);
-                               specs[6] = ItemSpec(ITEM_MATERIAL, CONTENT_COBBLE);
-                               specs[7] = ItemSpec(ITEM_MATERIAL, CONTENT_COBBLE);
-                               specs[8] = ItemSpec(ITEM_MATERIAL, CONTENT_COBBLE);
-                               if(checkItemCombination(items, specs))
-                               {
-                                       rlist->addItem(new MaterialItem(CONTENT_FURNACE, 1));
-                                       found = true;
-                               }
-                       }
-
-                       // Steel block
-                       if(!found)
-                       {
-                               ItemSpec specs[9];
-                               specs[0] = ItemSpec(ITEM_CRAFT, "steel_ingot");
-                               specs[1] = ItemSpec(ITEM_CRAFT, "steel_ingot");
-                               specs[2] = ItemSpec(ITEM_CRAFT, "steel_ingot");
-                               specs[3] = ItemSpec(ITEM_CRAFT, "steel_ingot");
-                               specs[4] = ItemSpec(ITEM_CRAFT, "steel_ingot");
-                               specs[5] = ItemSpec(ITEM_CRAFT, "steel_ingot");
-                               specs[6] = ItemSpec(ITEM_CRAFT, "steel_ingot");
-                               specs[7] = ItemSpec(ITEM_CRAFT, "steel_ingot");
-                               specs[8] = ItemSpec(ITEM_CRAFT, "steel_ingot");
-                               if(checkItemCombination(items, specs))
-                               {
-                                       rlist->addItem(new MaterialItem(CONTENT_STEEL, 1));
-                                       found = true;
-                               }
-                       }
+                       // Get result of crafting grid
+                       InventoryItem *result = craft_get_result(items);
+                       if(result)
+                               rlist->addItem(result);
                }
        
        } // if creative_mode == false
@@ -4201,108 +4166,25 @@ std::wstring Server::getStatusString()
        }
        os<<L"}";
        if(((ServerMap*)(&m_env.getMap()))->isSavingEnabled() == false)
-               os<<" WARNING: Map saving is disabled."<<std::endl;
+               os<<std::endl<<L"# Server: "<<" WARNING: Map saving is disabled.";
+       if(g_settings.get("motd") != "")
+               os<<std::endl<<L"# Server: "<<narrow_to_wide(g_settings.get("motd"));
        return os.str();
 }
 
-
-void setCreativeInventory(Player *player)
-{
-       player->resetInventory();
-       
-       // Give some good tools
-       {
-               InventoryItem *item = new ToolItem("MesePick", 0);
-               void* r = player->inventory.addItem("main", item);
-               assert(r == NULL);
-       }
-       {
-               InventoryItem *item = new ToolItem("SteelPick", 0);
-               void* r = player->inventory.addItem("main", item);
-               assert(r == NULL);
-       }
-       {
-               InventoryItem *item = new ToolItem("SteelAxe", 0);
-               void* r = player->inventory.addItem("main", item);
-               assert(r == NULL);
-       }
-       {
-               InventoryItem *item = new ToolItem("SteelShovel", 0);
-               void* r = player->inventory.addItem("main", item);
-               assert(r == NULL);
-       }
-
-       /*
-               Give materials
-       */
-       
-       // CONTENT_IGNORE-terminated list
-       u8 material_items[] = {
-               CONTENT_TORCH,
-               CONTENT_COBBLE,
-               CONTENT_MUD,
-               CONTENT_STONE,
-               CONTENT_SAND,
-               CONTENT_TREE,
-               CONTENT_LEAVES,
-               CONTENT_GLASS,
-               CONTENT_FENCE,
-               CONTENT_MESE,
-               CONTENT_WATERSOURCE,
-               CONTENT_CLOUD,
-               CONTENT_CHEST,
-               CONTENT_FURNACE,
-               CONTENT_SIGN_WALL,
-               CONTENT_IGNORE
-       };
-       
-       u8 *mip = material_items;
-       for(u16 i=0; i<PLAYER_INVENTORY_SIZE; i++)
-       {
-               if(*mip == CONTENT_IGNORE)
-                       break;
-
-               InventoryItem *item = new MaterialItem(*mip, 1);
-               player->inventory.addItem("main", item);
-
-               mip++;
-       }
-
-#if 0
-       assert(USEFUL_CONTENT_COUNT <= PLAYER_INVENTORY_SIZE);
-       
-       // add torch first
-       InventoryItem *item = new MaterialItem(CONTENT_TORCH, 1);
-       player->inventory.addItem("main", item);
-       
-       // Then others
-       for(u16 i=0; i<USEFUL_CONTENT_COUNT; i++)
-       {
-               // Skip some materials
-               if(i == CONTENT_WATER || i == CONTENT_TORCH
-                       || i == CONTENT_COALSTONE)
-                       continue;
-
-               InventoryItem *item = new MaterialItem(i, 1);
-               player->inventory.addItem("main", item);
-       }
-#endif
-
-       /*// Sign
-       {
-               InventoryItem *item = new MapBlockObjectItem("Sign Example text");
-               void* r = player->inventory.addItem("main", item);
-               assert(r == NULL);
-       }*/
-}
-
 v3f findSpawnPos(ServerMap &map)
 {
        //return v3f(50,50,50)*BS;
-       
+
        v2s16 nodepos;
        s16 groundheight = 0;
        
+#if 0
+       nodepos = v2s16(0,0);
+       groundheight = 20;
+#endif
+
+#if 1
        // Try to find a good place a few times
        for(s32 i=0; i<1000; i++)
        {
@@ -4332,6 +4214,7 @@ v3f findSpawnPos(ServerMap &map)
                //dstream<<"Searched through "<<i<<" places."<<std::endl;
                break;
        }
+#endif
        
        // If no suitable place was not found, go above water at least.
        if(groundheight < WATER_LEVEL)
@@ -4339,7 +4222,7 @@ v3f findSpawnPos(ServerMap &map)
 
        return intToFloat(v3s16(
                        nodepos.X,
-                       groundheight + 2,
+                       groundheight + 3,
                        nodepos.Y
                        ), BS);
 }
@@ -4365,7 +4248,12 @@ Player *Server::emergePlayer(const char *name, const char *password, u16 peer_id
                // Reset inventory to creative if in creative mode
                if(g_settings.getBool("creative_mode"))
                {
-                       setCreativeInventory(player);
+                       // Warning: double code below
+                       // Backup actual inventory
+                       player->inventory_backup = new Inventory();
+                       *(player->inventory_backup) = player->inventory;
+                       // Set creative inventory
+                       craft_set_creative_inventory(player);
                }
 
                return player;
@@ -4418,78 +4306,16 @@ Player *Server::emergePlayer(const char *name, const char *password, u16 peer_id
                
                if(g_settings.getBool("creative_mode"))
                {
-                       setCreativeInventory(player);
+                       // Warning: double code above
+                       // Backup actual inventory
+                       player->inventory_backup = new Inventory();
+                       *(player->inventory_backup) = player->inventory;
+                       // Set creative inventory
+                       craft_set_creative_inventory(player);
                }
                else if(g_settings.getBool("give_initial_stuff"))
                {
-                       {
-                               InventoryItem *item = new ToolItem("SteelPick", 0);
-                               void* r = player->inventory.addItem("main", item);
-                               assert(r == NULL);
-                       }
-                       {
-                               InventoryItem *item = new MaterialItem(CONTENT_TORCH, 99);
-                               void* r = player->inventory.addItem("main", item);
-                               assert(r == NULL);
-                       }
-                       {
-                               InventoryItem *item = new ToolItem("SteelAxe", 0);
-                               void* r = player->inventory.addItem("main", item);
-                               assert(r == NULL);
-                       }
-                       {
-                               InventoryItem *item = new ToolItem("SteelShovel", 0);
-                               void* r = player->inventory.addItem("main", item);
-                               assert(r == NULL);
-                       }
-                       {
-                               InventoryItem *item = new MaterialItem(CONTENT_COBBLE, 99);
-                               void* r = player->inventory.addItem("main", item);
-                               assert(r == NULL);
-                       }
-                       /*{
-                               InventoryItem *item = new MaterialItem(CONTENT_MESE, 6);
-                               void* r = player->inventory.addItem("main", item);
-                               assert(r == NULL);
-                       }
-                       {
-                               InventoryItem *item = new MaterialItem(CONTENT_COALSTONE, 6);
-                               void* r = player->inventory.addItem("main", item);
-                               assert(r == NULL);
-                       }
-                       {
-                               InventoryItem *item = new MaterialItem(CONTENT_WOOD, 6);
-                               void* r = player->inventory.addItem("main", item);
-                               assert(r == NULL);
-                       }
-                       {
-                               InventoryItem *item = new CraftItem("Stick", 4);
-                               void* r = player->inventory.addItem("main", item);
-                               assert(r == NULL);
-                       }
-                       {
-                               InventoryItem *item = new ToolItem("WPick", 32000);
-                               void* r = player->inventory.addItem("main", item);
-                               assert(r == NULL);
-                       }
-                       {
-                               InventoryItem *item = new ToolItem("STPick", 32000);
-                               void* r = player->inventory.addItem("main", item);
-                               assert(r == NULL);
-                       }*/
-                       /*// and some signs
-                       for(u16 i=0; i<4; i++)
-                       {
-                               InventoryItem *item = new MapBlockObjectItem("Sign Example text");
-                               bool r = player->inventory.addItem("main", item);
-                               assert(r == true);
-                       }*/
-                       /*// Give some other stuff
-                       {
-                               InventoryItem *item = new MaterialItem(CONTENT_TREE, 999);
-                               bool r = player->inventory.addItem("main", item);
-                               assert(r == true);
-                       }*/
+                       craft_give_initial_stuff(player);
                }
 
                return player;