]> git.lizzy.rs Git - dragonfireclient.git/blobdiff - src/map.cpp
map.cpp: Initialize NodeNeighbor, set NeighborType to u8 & cleanup
[dragonfireclient.git] / src / map.cpp
index ffba772620e6576a7f777b5066dff6a192f13323..054338c926653d84d0558797e7ebfd64fa797cc5 100644 (file)
@@ -37,23 +37,24 @@ with this program; if not, write to the Free Software Foundation, Inc.,
 #include "environment.h"
 #include "reflowscan.h"
 #include "emerge.h"
-#include "mapgen_v6.h"
-#include "mg_biome.h"
+#include "mapgen/mapgen_v6.h"
+#include "mapgen/mg_biome.h"
 #include "config.h"
 #include "server.h"
-#include "database.h"
-#include "database-dummy.h"
-#include "database-sqlite3.h"
+#include "database/database.h"
+#include "database/database-dummy.h"
+#include "database/database-sqlite3.h"
+#include "script/scripting_server.h"
 #include <deque>
 #include <queue>
 #if USE_LEVELDB
-#include "database-leveldb.h"
+#include "database/database-leveldb.h"
 #endif
 #if USE_REDIS
-#include "database-redis.h"
+#include "database/database-redis.h"
 #endif
 #if USE_POSTGRESQL
-#include "database-postgresql.h"
+#include "database/database-postgresql.h"
 #endif
 
 
@@ -64,12 +65,7 @@ with this program; if not, write to the Free Software Foundation, Inc.,
 Map::Map(std::ostream &dout, IGameDef *gamedef):
        m_dout(dout),
        m_gamedef(gamedef),
-       m_sector_cache(NULL),
-       m_nodedef(gamedef->ndef()),
-       m_transforming_liquid_loop_count_multiplier(1.0f),
-       m_unprocessed_count(0),
-       m_inc_trending_up_start_time(0),
-       m_queue_size_timer_started(false)
+       m_nodedef(gamedef->ndef())
 {
 }
 
@@ -78,10 +74,8 @@ Map::~Map()
        /*
                Free all MapSectors
        */
-       for(std::map<v2s16, MapSector*>::iterator i = m_sectors.begin();
-               i != m_sectors.end(); ++i)
-       {
-               delete i->second;
+       for (auto &sector : m_sectors) {
+               delete sector.second;
        }
 }
 
@@ -97,11 +91,8 @@ void Map::removeEventReceiver(MapEventReceiver *event_receiver)
 
 void Map::dispatchEvent(MapEditEvent *event)
 {
-       for(std::set<MapEventReceiver*>::iterator
-                       i = m_event_receivers.begin();
-                       i != m_event_receivers.end(); ++i)
-       {
-               (*i)->onMapEditEvent(event);
+       for (MapEventReceiver *event_receiver : m_event_receivers) {
+               event_receiver->onMapEditEvent(event);
        }
 }
 
@@ -114,7 +105,7 @@ MapSector * Map::getSectorNoGenerateNoExNoLock(v2s16 p)
 
        std::map<v2s16, MapSector*>::iterator n = m_sectors.find(p);
 
-       if(n == m_sectors.end())
+       if (n == m_sectors.end())
                return NULL;
 
        MapSector *sector = n->second;
@@ -186,7 +177,7 @@ MapNode Map::getNodeNoEx(v3s16 p, bool *is_valid_position)
        if (block == NULL) {
                if (is_valid_position != NULL)
                        *is_valid_position = false;
-               return MapNode(CONTENT_IGNORE);
+               return {CONTENT_IGNORE};
        }
 
        v3s16 relpos = p - blockpos*MAP_BLOCKSIZE;
@@ -229,577 +220,11 @@ void Map::setNode(v3s16 p, MapNode & n)
                                <<" while trying to replace \""
                                <<m_nodedef->get(block->getNodeNoCheck(relpos, &temp_bool)).name
                                <<"\" at "<<PP(p)<<" (block "<<PP(blockpos)<<")"<<std::endl;
-               debug_stacks_print_to(infostream);
                return;
        }
        block->setNodeNoCheck(relpos, n);
 }
 
-/*
-       Goes recursively through the neighbours of the node.
-
-       Alters only transparent nodes.
-
-       If the lighting of the neighbour is lower than the lighting of
-       the node was (before changing it to 0 at the step before), the
-       lighting of the neighbour is set to 0 and then the same stuff
-       repeats for the neighbour.
-
-       The ending nodes of the routine are stored in light_sources.
-       This is useful when a light is removed. In such case, this
-       routine can be called for the light node and then again for
-       light_sources to re-light the area without the removed light.
-
-       values of from_nodes are lighting values.
-*/
-void Map::unspreadLight(enum LightBank bank,
-               std::map<v3s16, u8> & from_nodes,
-               std::set<v3s16> & light_sources,
-               std::map<v3s16, MapBlock*>  & modified_blocks)
-{
-       v3s16 dirs[6] = {
-               v3s16(0,0,1), // back
-               v3s16(0,1,0), // top
-               v3s16(1,0,0), // right
-               v3s16(0,0,-1), // front
-               v3s16(0,-1,0), // bottom
-               v3s16(-1,0,0), // left
-       };
-
-       if(from_nodes.empty())
-               return;
-
-       u32 blockchangecount = 0;
-
-       std::map<v3s16, u8> unlighted_nodes;
-
-       /*
-               Initialize block cache
-       */
-       v3s16 blockpos_last;
-       MapBlock *block = NULL;
-       // Cache this a bit, too
-       bool block_checked_in_modified = false;
-
-       for(std::map<v3s16, u8>::iterator j = from_nodes.begin();
-               j != from_nodes.end(); ++j)
-       {
-               v3s16 pos = j->first;
-               v3s16 blockpos = getNodeBlockPos(pos);
-
-               // Only fetch a new block if the block position has changed
-               try{
-                       if(block == NULL || blockpos != blockpos_last){
-                               block = getBlockNoCreate(blockpos);
-                               blockpos_last = blockpos;
-
-                               block_checked_in_modified = false;
-                               blockchangecount++;
-                       }
-               }
-               catch(InvalidPositionException &e)
-               {
-                       continue;
-               }
-
-               if(block->isDummy())
-                       continue;
-
-               // Calculate relative position in block
-               //v3s16 relpos = pos - blockpos_last * MAP_BLOCKSIZE;
-
-               // Get node straight from the block
-               //MapNode n = block->getNode(relpos);
-
-               u8 oldlight = j->second;
-
-               // Loop through 6 neighbors
-               for(u16 i=0; i<6; i++)
-               {
-                       // Get the position of the neighbor node
-                       v3s16 n2pos = pos + dirs[i];
-
-                       // Get the block where the node is located
-                       v3s16 blockpos, relpos;
-                       getNodeBlockPosWithOffset(n2pos, blockpos, relpos);
-
-                       // Only fetch a new block if the block position has changed
-                       try {
-                               if(block == NULL || blockpos != blockpos_last){
-                                       block = getBlockNoCreate(blockpos);
-                                       blockpos_last = blockpos;
-
-                                       block_checked_in_modified = false;
-                                       blockchangecount++;
-                               }
-                       }
-                       catch(InvalidPositionException &e) {
-                               continue;
-                       }
-
-                       // Get node straight from the block
-                       bool is_valid_position;
-                       MapNode n2 = block->getNode(relpos, &is_valid_position);
-                       if (!is_valid_position)
-                               continue;
-
-                       bool changed = false;
-
-                       //TODO: Optimize output by optimizing light_sources?
-
-                       /*
-                               If the neighbor is dimmer than what was specified
-                               as oldlight (the light of the previous node)
-                       */
-                       if(n2.getLight(bank, m_nodedef) < oldlight)
-                       {
-                               /*
-                                       And the neighbor is transparent and it has some light
-                               */
-                               if(m_nodedef->get(n2).light_propagates
-                                               && n2.getLight(bank, m_nodedef) != 0)
-                               {
-                                       /*
-                                               Set light to 0 and add to queue
-                                       */
-
-                                       u8 current_light = n2.getLight(bank, m_nodedef);
-                                       n2.setLight(bank, 0, m_nodedef);
-                                       block->setNode(relpos, n2);
-
-                                       unlighted_nodes[n2pos] = current_light;
-                                       changed = true;
-
-                                       /*
-                                               Remove from light_sources if it is there
-                                               NOTE: This doesn't happen nearly at all
-                                       */
-                                       /*if(light_sources.find(n2pos))
-                                       {
-                                               infostream<<"Removed from light_sources"<<std::endl;
-                                               light_sources.remove(n2pos);
-                                       }*/
-                               }
-
-                               /*// DEBUG
-                               if(light_sources.find(n2pos) != NULL)
-                                       light_sources.remove(n2pos);*/
-                       }
-                       else{
-                               light_sources.insert(n2pos);
-                       }
-
-                       // Add to modified_blocks
-                       if(changed == true && block_checked_in_modified == false)
-                       {
-                               // If the block is not found in modified_blocks, add.
-                               if(modified_blocks.find(blockpos) == modified_blocks.end())
-                               {
-                                       modified_blocks[blockpos] = block;
-                               }
-                               block_checked_in_modified = true;
-                       }
-               }
-       }
-
-       /*infostream<<"unspreadLight(): Changed block "
-       <<blockchangecount<<" times"
-       <<" for "<<from_nodes.size()<<" nodes"
-       <<std::endl;*/
-
-       if(!unlighted_nodes.empty())
-               unspreadLight(bank, unlighted_nodes, light_sources, modified_blocks);
-}
-
-/*
-       Lights neighbors of from_nodes, collects all them and then
-       goes on recursively.
-*/
-void Map::spreadLight(enum LightBank bank,
-               std::set<v3s16> & from_nodes,
-               std::map<v3s16, MapBlock*> & modified_blocks)
-{
-       const v3s16 dirs[6] = {
-               v3s16(0,0,1), // back
-               v3s16(0,1,0), // top
-               v3s16(1,0,0), // right
-               v3s16(0,0,-1), // front
-               v3s16(0,-1,0), // bottom
-               v3s16(-1,0,0), // left
-       };
-
-       if(from_nodes.empty())
-               return;
-
-       u32 blockchangecount = 0;
-
-       std::set<v3s16> lighted_nodes;
-
-       /*
-               Initialize block cache
-       */
-       v3s16 blockpos_last;
-       MapBlock *block = NULL;
-               // Cache this a bit, too
-       bool block_checked_in_modified = false;
-
-       for(std::set<v3s16>::iterator j = from_nodes.begin();
-               j != from_nodes.end(); ++j)
-       {
-               v3s16 pos = *j;
-               v3s16 blockpos, relpos;
-
-               getNodeBlockPosWithOffset(pos, blockpos, relpos);
-
-               // Only fetch a new block if the block position has changed
-               try {
-                       if(block == NULL || blockpos != blockpos_last){
-                               block = getBlockNoCreate(blockpos);
-                               blockpos_last = blockpos;
-
-                               block_checked_in_modified = false;
-                               blockchangecount++;
-                       }
-               }
-               catch(InvalidPositionException &e) {
-                       continue;
-               }
-
-               if(block->isDummy())
-                       continue;
-
-               // Get node straight from the block
-               bool is_valid_position;
-               MapNode n = block->getNode(relpos, &is_valid_position);
-
-               u8 oldlight = is_valid_position ? n.getLight(bank, m_nodedef) : 0;
-               u8 newlight = diminish_light(oldlight);
-
-               // Loop through 6 neighbors
-               for(u16 i=0; i<6; i++){
-                       // Get the position of the neighbor node
-                       v3s16 n2pos = pos + dirs[i];
-
-                       // Get the block where the node is located
-                       v3s16 blockpos, relpos;
-                       getNodeBlockPosWithOffset(n2pos, blockpos, relpos);
-
-                       // Only fetch a new block if the block position has changed
-                       try {
-                               if(block == NULL || blockpos != blockpos_last){
-                                       block = getBlockNoCreate(blockpos);
-                                       blockpos_last = blockpos;
-
-                                       block_checked_in_modified = false;
-                                       blockchangecount++;
-                               }
-                       }
-                       catch(InvalidPositionException &e) {
-                               continue;
-                       }
-
-                       // Get node straight from the block
-                       MapNode n2 = block->getNode(relpos, &is_valid_position);
-                       if (!is_valid_position)
-                               continue;
-
-                       bool changed = false;
-                       /*
-                               If the neighbor is brighter than the current node,
-                               add to list (it will light up this node on its turn)
-                       */
-                       if(n2.getLight(bank, m_nodedef) > undiminish_light(oldlight))
-                       {
-                               lighted_nodes.insert(n2pos);
-                               changed = true;
-                       }
-                       /*
-                               If the neighbor is dimmer than how much light this node
-                               would spread on it, add to list
-                       */
-                       if(n2.getLight(bank, m_nodedef) < newlight)
-                       {
-                               if(m_nodedef->get(n2).light_propagates)
-                               {
-                                       n2.setLight(bank, newlight, m_nodedef);
-                                       block->setNode(relpos, n2);
-                                       lighted_nodes.insert(n2pos);
-                                       changed = true;
-                               }
-                       }
-
-                       // Add to modified_blocks
-                       if(changed == true && block_checked_in_modified == false)
-                       {
-                               // If the block is not found in modified_blocks, add.
-                               if(modified_blocks.find(blockpos) == modified_blocks.end())
-                               {
-                                       modified_blocks[blockpos] = block;
-                               }
-                               block_checked_in_modified = true;
-                       }
-               }
-       }
-
-       /*infostream<<"spreadLight(): Changed block "
-                       <<blockchangecount<<" times"
-                       <<" for "<<from_nodes.size()<<" nodes"
-                       <<std::endl;*/
-
-       if(!lighted_nodes.empty())
-               spreadLight(bank, lighted_nodes, modified_blocks);
-}
-
-void Map::updateLighting(enum LightBank bank,
-               std::map<v3s16, MapBlock*> & a_blocks,
-               std::map<v3s16, MapBlock*> & modified_blocks)
-{
-       /*m_dout<<"Map::updateLighting(): "
-                       <<a_blocks.size()<<" blocks."<<std::endl;*/
-
-       //TimeTaker timer("updateLighting");
-
-       // For debugging
-       //bool debug=true;
-       //u32 count_was = modified_blocks.size();
-
-       //std::map<v3s16, MapBlock*> blocks_to_update;
-
-       std::set<v3s16> light_sources;
-
-       std::map<v3s16, u8> unlight_from;
-
-       int num_bottom_invalid = 0;
-
-       {
-       //TimeTaker t("first stuff");
-
-       for(std::map<v3s16, MapBlock*>::iterator i = a_blocks.begin();
-               i != a_blocks.end(); ++i)
-       {
-               MapBlock *block = i->second;
-
-               for(;;)
-               {
-                       // Don't bother with dummy blocks.
-                       if(block->isDummy())
-                               break;
-
-                       v3s16 pos = block->getPos();
-                       v3s16 posnodes = block->getPosRelative();
-                       modified_blocks[pos] = block;
-                       //blocks_to_update[pos] = block;
-
-                       /*
-                               Clear all light from block
-                       */
-                       for(s16 z=0; z<MAP_BLOCKSIZE; z++)
-                       for(s16 x=0; x<MAP_BLOCKSIZE; x++)
-                       for(s16 y=0; y<MAP_BLOCKSIZE; y++)
-                       {
-                               v3s16 p(x,y,z);
-                               bool is_valid_position;
-                               MapNode n = block->getNode(p, &is_valid_position);
-                               if (!is_valid_position) {
-                                       /* This would happen when dealing with a
-                                          dummy block.
-                                       */
-                                       infostream<<"updateLighting(): InvalidPositionException"
-                                                       <<std::endl;
-                                       continue;
-                               }
-                               u8 oldlight = n.getLight(bank, m_nodedef);
-                               n.setLight(bank, 0, m_nodedef);
-                               block->setNode(p, n);
-
-                               // If node sources light, add to list
-                               u8 source = m_nodedef->get(n).light_source;
-                               if(source != 0)
-                                       light_sources.insert(p + posnodes);
-
-                               // Collect borders for unlighting
-                               if((x==0 || x == MAP_BLOCKSIZE-1
-                                               || y==0 || y == MAP_BLOCKSIZE-1
-                                               || z==0 || z == MAP_BLOCKSIZE-1)
-                                               && oldlight != 0)
-                               {
-                                       v3s16 p_map = p + posnodes;
-                                       unlight_from[p_map] = oldlight;
-                               }
-
-
-                       }
-
-                       if(bank == LIGHTBANK_DAY)
-                       {
-                               bool bottom_valid = block->propagateSunlight(light_sources);
-
-                               if(!bottom_valid)
-                                       num_bottom_invalid++;
-
-                               // If bottom is valid, we're done.
-                               if(bottom_valid)
-                                       break;
-                       }
-                       else if(bank == LIGHTBANK_NIGHT)
-                       {
-                               // For night lighting, sunlight is not propagated
-                               break;
-                       }
-                       else
-                       {
-                               assert("Invalid lighting bank" == NULL);
-                       }
-
-                       /*infostream<<"Bottom for sunlight-propagated block ("
-                                       <<pos.X<<","<<pos.Y<<","<<pos.Z<<") not valid"
-                                       <<std::endl;*/
-
-                       // Bottom sunlight is not valid; get the block and loop to it
-
-                       pos.Y--;
-                       try{
-                               block = getBlockNoCreate(pos);
-                       }
-                       catch(InvalidPositionException &e)
-                       {
-                               FATAL_ERROR("Invalid position");
-                       }
-
-               }
-       }
-
-       }
-
-       /*
-               Enable this to disable proper lighting for speeding up map
-               generation for testing or whatever
-       */
-#if 0
-       //if(g_settings->get(""))
-       {
-               core::map<v3s16, MapBlock*>::Iterator i;
-               i = blocks_to_update.getIterator();
-               for(; i.atEnd() == false; i++)
-               {
-                       MapBlock *block = i.getNode()->getValue();
-                       v3s16 p = block->getPos();
-                       block->setLightingExpired(false);
-               }
-               return;
-       }
-#endif
-
-#if 1
-       {
-               //TimeTaker timer("unspreadLight");
-               unspreadLight(bank, unlight_from, light_sources, modified_blocks);
-       }
-
-       /*if(debug)
-       {
-               u32 diff = modified_blocks.size() - count_was;
-               count_was = modified_blocks.size();
-               infostream<<"unspreadLight modified "<<diff<<std::endl;
-       }*/
-
-       {
-               //TimeTaker timer("spreadLight");
-               spreadLight(bank, light_sources, modified_blocks);
-       }
-
-       /*if(debug)
-       {
-               u32 diff = modified_blocks.size() - count_was;
-               count_was = modified_blocks.size();
-               infostream<<"spreadLight modified "<<diff<<std::endl;
-       }*/
-#endif
-
-#if 0
-       {
-               //MapVoxelManipulator vmanip(this);
-
-               // Make a manual voxel manipulator and load all the blocks
-               // that touch the requested blocks
-               ManualMapVoxelManipulator vmanip(this);
-
-               {
-               //TimeTaker timer("initialEmerge");
-
-               core::map<v3s16, MapBlock*>::Iterator i;
-               i = blocks_to_update.getIterator();
-               for(; i.atEnd() == false; i++)
-               {
-                       MapBlock *block = i.getNode()->getValue();
-                       v3s16 p = block->getPos();
-
-                       // Add all surrounding blocks
-                       vmanip.initialEmerge(p - v3s16(1,1,1), p + v3s16(1,1,1));
-
-                       /*
-                               Add all surrounding blocks that have up-to-date lighting
-                               NOTE: This doesn't quite do the job (not everything
-                                         appropriate is lighted)
-                       */
-                       /*for(s16 z=-1; z<=1; z++)
-                       for(s16 y=-1; y<=1; y++)
-                       for(s16 x=-1; x<=1; x++)
-                       {
-                               v3s16 p2 = p + v3s16(x,y,z);
-                               MapBlock *block = getBlockNoCreateNoEx(p2);
-                               if(block == NULL)
-                                       continue;
-                               if(block->isDummy())
-                                       continue;
-                               if(block->getLightingExpired())
-                                       continue;
-                               vmanip.initialEmerge(p2, p2);
-                       }*/
-
-                       // Lighting of block will be updated completely
-                       block->setLightingExpired(false);
-               }
-               }
-
-               {
-                       //TimeTaker timer("unSpreadLight");
-                       vmanip.unspreadLight(bank, unlight_from, light_sources, nodemgr);
-               }
-               {
-                       //TimeTaker timer("spreadLight");
-                       vmanip.spreadLight(bank, light_sources, nodemgr);
-               }
-               {
-                       //TimeTaker timer("blitBack");
-                       vmanip.blitBack(modified_blocks);
-               }
-               /*infostream<<"emerge_time="<<emerge_time<<std::endl;
-               emerge_time = 0;*/
-       }
-#endif
-
-       //m_dout<<"Done ("<<getTimestamp()<<")"<<std::endl;
-}
-
-void Map::updateLighting(std::map<v3s16, MapBlock*> & a_blocks,
-               std::map<v3s16, MapBlock*> & modified_blocks)
-{
-       updateLighting(LIGHTBANK_DAY, a_blocks, modified_blocks);
-       updateLighting(LIGHTBANK_NIGHT, a_blocks, modified_blocks);
-
-       /*
-               Update information about whether day and night light differ
-       */
-       for(std::map<v3s16, MapBlock*>::iterator
-                       i = modified_blocks.begin();
-                       i != modified_blocks.end(); ++i)
-       {
-               MapBlock *block = i->second;
-               block->expireDayNightDiff();
-       }
-}
-
 void Map::addNodeAndUpdate(v3s16 p, MapNode n,
                std::map<v3s16, MapBlock*> &modified_blocks,
                bool remove_metadata)
@@ -823,14 +248,11 @@ void Map::addNodeAndUpdate(v3s16 p, MapNode n,
 
        // Update lighting
        std::vector<std::pair<v3s16, MapNode> > oldnodes;
-       oldnodes.push_back(std::pair<v3s16, MapNode>(p, oldnode));
+       oldnodes.emplace_back(p, oldnode);
        voxalgo::update_lighting_nodes(this, oldnodes, modified_blocks);
 
-       for(std::map<v3s16, MapBlock*>::iterator
-                       i = modified_blocks.begin();
-                       i != modified_blocks.end(); ++i)
-       {
-               i->second->expireDayNightDiff();
+       for (auto &modified_block : modified_blocks) {
+               modified_block.second->expireDayNightDiff();
        }
 
        // Report for rollback
@@ -846,18 +268,9 @@ void Map::addNodeAndUpdate(v3s16 p, MapNode n,
                Add neighboring liquid nodes and this node to transform queue.
                (it's vital for the node itself to get updated last, if it was removed.)
         */
-       v3s16 dirs[7] = {
-               v3s16(0,0,1), // back
-               v3s16(0,1,0), // top
-               v3s16(1,0,0), // right
-               v3s16(0,0,-1), // front
-               v3s16(0,-1,0), // bottom
-               v3s16(-1,0,0), // left
-               v3s16(0,0,0), // self
-       };
-       for(u16 i=0; i<7; i++)
-       {
-               v3s16 p2 = p + dirs[i];
+
+       for (const v3s16 &dir : g_7dirs) {
+               v3s16 p2 = p + dir;
 
                bool is_valid_position;
                MapNode n2 = getNodeNoEx(p2, &is_valid_position);
@@ -887,11 +300,8 @@ bool Map::addNodeWithEvent(v3s16 p, MapNode n, bool remove_metadata)
                addNodeAndUpdate(p, n, modified_blocks, remove_metadata);
 
                // Copy modified_blocks to event
-               for(std::map<v3s16, MapBlock*>::iterator
-                               i = modified_blocks.begin();
-                               i != modified_blocks.end(); ++i)
-               {
-                       event.modified_blocks.insert(i->first);
+               for (auto &modified_block : modified_blocks) {
+                       event.modified_blocks.insert(modified_block.first);
                }
        }
        catch(InvalidPositionException &e){
@@ -915,11 +325,8 @@ bool Map::removeNodeWithEvent(v3s16 p)
                removeNodeAndUpdate(p, modified_blocks);
 
                // Copy modified_blocks to event
-               for(std::map<v3s16, MapBlock*>::iterator
-                               i = modified_blocks.begin();
-                               i != modified_blocks.end(); ++i)
-               {
-                       event.modified_blocks.insert(i->first);
+               for (auto &modified_block : modified_blocks) {
+                       event.modified_blocks.insert(modified_block.first);
                }
        }
        catch(InvalidPositionException &e){
@@ -931,63 +338,6 @@ bool Map::removeNodeWithEvent(v3s16 p)
        return succeeded;
 }
 
-bool Map::getDayNightDiff(v3s16 blockpos)
-{
-       try{
-               v3s16 p = blockpos + v3s16(0,0,0);
-               MapBlock *b = getBlockNoCreate(p);
-               if(b->getDayNightDiff())
-                       return true;
-       }
-       catch(InvalidPositionException &e){}
-       // Leading edges
-       try{
-               v3s16 p = blockpos + v3s16(-1,0,0);
-               MapBlock *b = getBlockNoCreate(p);
-               if(b->getDayNightDiff())
-                       return true;
-       }
-       catch(InvalidPositionException &e){}
-       try{
-               v3s16 p = blockpos + v3s16(0,-1,0);
-               MapBlock *b = getBlockNoCreate(p);
-               if(b->getDayNightDiff())
-                       return true;
-       }
-       catch(InvalidPositionException &e){}
-       try{
-               v3s16 p = blockpos + v3s16(0,0,-1);
-               MapBlock *b = getBlockNoCreate(p);
-               if(b->getDayNightDiff())
-                       return true;
-       }
-       catch(InvalidPositionException &e){}
-       // Trailing edges
-       try{
-               v3s16 p = blockpos + v3s16(1,0,0);
-               MapBlock *b = getBlockNoCreate(p);
-               if(b->getDayNightDiff())
-                       return true;
-       }
-       catch(InvalidPositionException &e){}
-       try{
-               v3s16 p = blockpos + v3s16(0,1,0);
-               MapBlock *b = getBlockNoCreate(p);
-               if(b->getDayNightDiff())
-                       return true;
-       }
-       catch(InvalidPositionException &e){}
-       try{
-               v3s16 p = blockpos + v3s16(0,0,1);
-               MapBlock *b = getBlockNoCreate(p);
-               if(b->getDayNightDiff())
-                       return true;
-       }
-       catch(InvalidPositionException &e){}
-
-       return false;
-}
-
 struct TimeOrderedMapBlock {
        MapSector *sect;
        MapBlock *block;
@@ -1023,19 +373,15 @@ void Map::timerUpdate(float dtime, float unload_timeout, u32 max_loaded_blocks,
 
        // If there is no practical limit, we spare creation of mapblock_queue
        if (max_loaded_blocks == U32_MAX) {
-               for (std::map<v2s16, MapSector*>::iterator si = m_sectors.begin();
-                               si != m_sectors.end(); ++si) {
-                       MapSector *sector = si->second;
+               for (auto &sector_it : m_sectors) {
+                       MapSector *sector = sector_it.second;
 
                        bool all_blocks_deleted = true;
 
                        MapBlockVect blocks;
                        sector->getBlocks(blocks);
 
-                       for (MapBlockVect::iterator i = blocks.begin();
-                                       i != blocks.end(); ++i) {
-                               MapBlock *block = (*i);
-
+                       for (MapBlock *block : blocks) {
                                block->incrementUsageTimer(dtime);
 
                                if (block->refGet() == 0
@@ -1065,22 +411,18 @@ void Map::timerUpdate(float dtime, float unload_timeout, u32 max_loaded_blocks,
                        }
 
                        if (all_blocks_deleted) {
-                               sector_deletion_queue.push_back(si->first);
+                               sector_deletion_queue.push_back(sector_it.first);
                        }
                }
        } else {
                std::priority_queue<TimeOrderedMapBlock> mapblock_queue;
-               for (std::map<v2s16, MapSector*>::iterator si = m_sectors.begin();
-                               si != m_sectors.end(); ++si) {
-                       MapSector *sector = si->second;
+               for (auto &sector_it : m_sectors) {
+                       MapSector *sector = sector_it.second;
 
                        MapBlockVect blocks;
                        sector->getBlocks(blocks);
 
-                       for(MapBlockVect::iterator i = blocks.begin();
-                                       i != blocks.end(); ++i) {
-                               MapBlock *block = (*i);
-
+                       for (MapBlock *block : blocks) {
                                block->incrementUsageTimer(dtime);
                                mapblock_queue.push(TimeOrderedMapBlock(sector, block));
                        }
@@ -1117,10 +459,9 @@ void Map::timerUpdate(float dtime, float unload_timeout, u32 max_loaded_blocks,
                        block_count_all--;
                }
                // Delete empty sectors
-               for (std::map<v2s16, MapSector*>::iterator si = m_sectors.begin();
-                       si != m_sectors.end(); ++si) {
-                       if (si->second->empty()) {
-                               sector_deletion_queue.push_back(si->first);
+               for (auto &sector_it : m_sectors) {
+                       if (sector_it.second->empty()) {
+                               sector_deletion_queue.push_back(sector_it.first);
                        }
                }
        }
@@ -1153,14 +494,13 @@ void Map::unloadUnreferencedBlocks(std::vector<v3s16> *unloaded_blocks)
 
 void Map::deleteSectors(std::vector<v2s16> &sectorList)
 {
-       for(std::vector<v2s16>::iterator j = sectorList.begin();
-               j != sectorList.end(); ++j) {
-               MapSector *sector = m_sectors[*j];
+       for (v2s16 j : sectorList) {
+               MapSector *sector = m_sectors[j];
                // If sector is in sector cache, remove it from there
                if(m_sector_cache == sector)
                        m_sector_cache = NULL;
                // Remove from map and delete
-               m_sectors.erase(*j);
+               m_sectors.erase(j);
                delete sector;
        }
 }
@@ -1172,22 +512,22 @@ void Map::PrintInfo(std::ostream &out)
 
 #define WATER_DROP_BOOST 4
 
-enum NeighborType {
+enum NeighborType : u8 {
        NEIGHBOR_UPPER,
        NEIGHBOR_SAME_LEVEL,
        NEIGHBOR_LOWER
 };
+
 struct NodeNeighbor {
        MapNode n;
        NeighborType t;
        v3s16 p;
-       bool l; //can liquid
 
        NodeNeighbor()
-               : n(CONTENT_AIR)
+               : n(CONTENT_AIR), t(NEIGHBOR_SAME_LEVEL)
        { }
 
-       NodeNeighbor(const MapNode &node, NeighborType n_type, v3s16 pos)
+       NodeNeighbor(const MapNode &node, NeighborType n_type, const v3s16 &pos)
                : n(node),
                  t(n_type),
                  p(pos)
@@ -1198,15 +538,9 @@ void Map::transforming_liquid_add(v3s16 p) {
         m_transforming_liquid.push_back(p);
 }
 
-s32 Map::transforming_liquid_size() {
-        return m_transforming_liquid.size();
-}
-
-void Map::transformLiquids(std::map<v3s16, MapBlock*> &modified_blocks)
+void Map::transformLiquids(std::map<v3s16, MapBlock*> &modified_blocks,
+               ServerEnvironment *env)
 {
-       DSTACK(FUNCTION_NAME);
-       //TimeTaker timer("transformLiquids()");
-
        u32 loopcount = 0;
        u32 initial_size = m_transforming_liquid.size();
 
@@ -1307,6 +641,8 @@ void Map::transformLiquids(std::map<v3s16, MapBlock*> &modified_blocks)
                                case 4:
                                        nt = NEIGHBOR_LOWER;
                                        break;
+                               default:
+                                       break;
                        }
                        v3s16 npos = p0 + dirs[i];
                        NodeNeighbor nb(getNodeNoEx(npos), nt, npos);
@@ -1462,8 +798,16 @@ void Map::transformLiquids(std::map<v3s16, MapBlock*> &modified_blocks)
                        // set the liquid level and flow bit to 0
                        n0.param2 = ~(LIQUID_LEVEL_MASK | LIQUID_FLOW_DOWN_MASK);
                }
+
+               // change the node.
                n0.setContent(new_node_content);
 
+               // on_flood() the node
+               if (floodable_node != CONTENT_AIR) {
+                       if (env->getScriptIface()->node_on_flood(p0, n00, n0))
+                               continue;
+               }
+
                // Ignore light (because calling voxalgo::update_lighting_nodes)
                n0.setLight(LIGHTBANK_DAY, 0, m_nodedef);
                n0.setLight(LIGHTBANK_NIGHT, 0, m_nodedef);
@@ -1494,7 +838,7 @@ void Map::transformLiquids(std::map<v3s16, MapBlock*> &modified_blocks)
                MapBlock *block = getBlockNoCreateNoEx(blockpos);
                if (block != NULL) {
                        modified_blocks[blockpos] =  block;
-                       changed_nodes.push_back(std::pair<v3s16, MapNode>(p0, n00));
+                       changed_nodes.emplace_back(p0, n00);
                }
 
                /*
@@ -1520,8 +864,8 @@ void Map::transformLiquids(std::map<v3s16, MapBlock*> &modified_blocks)
        }
        //infostream<<"Map::transformLiquids(): loopcount="<<loopcount<<std::endl;
 
-       for (std::deque<v3s16>::iterator iter = must_reflow.begin(); iter != must_reflow.end(); ++iter)
-               m_transforming_liquid.push_back(*iter);
+       for (auto &iter : must_reflow)
+               m_transforming_liquid.push_back(iter);
 
        voxalgo::update_lighting_nodes(this, changed_nodes, modified_blocks);
 
@@ -1536,7 +880,7 @@ void Map::transformLiquids(std::map<v3s16, MapBlock*> &modified_blocks)
 
        time_until_purge *= 1000;       // seconds -> milliseconds
 
-       u32 curr_time = getTime(PRECISION_MILLI);
+       u64 curr_time = porting::getTimeMs();
        u32 prev_unprocessed = m_unprocessed_count;
        m_unprocessed_count = m_transforming_liquid.size();
 
@@ -1722,14 +1066,79 @@ void Map::removeNodeTimer(v3s16 p)
        block->m_node_timers.remove(p_rel);
 }
 
+bool Map::isOccluded(v3s16 p0, v3s16 p1, float step, float stepfac,
+               float start_off, float end_off, u32 needed_count)
+{
+       float d0 = (float)BS * p0.getDistanceFrom(p1);
+       v3s16 u0 = p1 - p0;
+       v3f uf = v3f(u0.X, u0.Y, u0.Z) * BS;
+       uf.normalize();
+       v3f p0f = v3f(p0.X, p0.Y, p0.Z) * BS;
+       u32 count = 0;
+       for(float s=start_off; s<d0+end_off; s+=step){
+               v3f pf = p0f + uf * s;
+               v3s16 p = floatToInt(pf, BS);
+               MapNode n = getNodeNoEx(p);
+               const ContentFeatures &f = m_nodedef->get(n);
+               if(f.drawtype == NDT_NORMAL){
+                       // not transparent, see ContentFeature::updateTextures
+                       count++;
+                       if(count >= needed_count)
+                               return true;
+               }
+               step *= stepfac;
+       }
+       return false;
+}
+
+bool Map::isBlockOccluded(MapBlock *block, v3s16 cam_pos_nodes) {
+       v3s16 cpn = block->getPos() * MAP_BLOCKSIZE;
+       cpn += v3s16(MAP_BLOCKSIZE / 2, MAP_BLOCKSIZE / 2, MAP_BLOCKSIZE / 2);
+       float step = BS * 1;
+       float stepfac = 1.1;
+       float startoff = BS * 1;
+       // The occlusion search of 'isOccluded()' must stop short of the target
+       // point by distance 'endoff' (end offset) to not enter the target mapblock.
+       // For the 8 mapblock corners 'endoff' must therefore be the maximum diagonal
+       // of a mapblock, because we must consider all view angles.
+       // sqrt(1^2 + 1^2 + 1^2) = 1.732
+       float endoff = -BS * MAP_BLOCKSIZE * 1.732050807569;
+       s16 bs2 = MAP_BLOCKSIZE / 2 + 1;
+       // to reduce the likelihood of falsely occluded blocks
+       // require at least two solid blocks
+       // this is a HACK, we should think of a more precise algorithm
+       u32 needed_count = 2;
+
+       return (
+               // For the central point of the mapblock 'endoff' can be halved
+               isOccluded(cam_pos_nodes, cpn,
+                       step, stepfac, startoff, endoff / 2.0f, needed_count) &&
+               isOccluded(cam_pos_nodes, cpn + v3s16(bs2,bs2,bs2),
+                       step, stepfac, startoff, endoff, needed_count) &&
+               isOccluded(cam_pos_nodes, cpn + v3s16(bs2,bs2,-bs2),
+                       step, stepfac, startoff, endoff, needed_count) &&
+               isOccluded(cam_pos_nodes, cpn + v3s16(bs2,-bs2,bs2),
+                       step, stepfac, startoff, endoff, needed_count) &&
+               isOccluded(cam_pos_nodes, cpn + v3s16(bs2,-bs2,-bs2),
+                       step, stepfac, startoff, endoff, needed_count) &&
+               isOccluded(cam_pos_nodes, cpn + v3s16(-bs2,bs2,bs2),
+                       step, stepfac, startoff, endoff, needed_count) &&
+               isOccluded(cam_pos_nodes, cpn + v3s16(-bs2,bs2,-bs2),
+                       step, stepfac, startoff, endoff, needed_count) &&
+               isOccluded(cam_pos_nodes, cpn + v3s16(-bs2,-bs2,bs2),
+                       step, stepfac, startoff, endoff, needed_count) &&
+               isOccluded(cam_pos_nodes, cpn + v3s16(-bs2,-bs2,-bs2),
+                       step, stepfac, startoff, endoff, needed_count));
+}
+
 /*
        ServerMap
 */
-ServerMap::ServerMap(std::string savedir, IGameDef *gamedef, EmergeManager *emerge):
+ServerMap::ServerMap(const std::string &savedir, IGameDef *gamedef,
+               EmergeManager *emerge):
        Map(dout_server, gamedef),
        settings_mgr(g_settings, savedir + DIR_DELIM + "map_meta.txt"),
-       m_emerge(emerge),
-       m_map_metadata_changed(true)
+       m_emerge(emerge)
 {
        verbosestream<<FUNCTION_NAME<<std::endl;
 
@@ -1757,14 +1166,11 @@ ServerMap::ServerMap(std::string savedir, IGameDef *gamedef, EmergeManager *emer
        m_savedir = savedir;
        m_map_saving_enabled = false;
 
-       try
-       {
+       try {
                // If directory exists, check contents and load if possible
-               if(fs::PathExists(m_savedir))
-               {
+               if (fs::PathExists(m_savedir)) {
                        // If directory is empty, it is safe to save into it.
-                       if(fs::GetDirListing(m_savedir).size() == 0)
-                       {
+                       if (fs::GetDirListing(m_savedir).empty()) {
                                infostream<<"ServerMap: Empty save directory is valid."
                                                <<std::endl;
                                m_map_saving_enabled = true;
@@ -1798,14 +1204,6 @@ ServerMap::ServerMap(std::string savedir, IGameDef *gamedef, EmergeManager *emer
                infostream<<"Please remove the map or fix it."<<std::endl;
                warningstream<<"Map saving will be disabled."<<std::endl;
        }
-
-       infostream<<"Initializing new map."<<std::endl;
-
-       // Create zero sector
-       emergeSector(v2s16(0,0));
-
-       // Initially write whole map
-       save(MOD_STATE_CLEAN);
 }
 
 ServerMap::~ServerMap()
@@ -1814,15 +1212,12 @@ ServerMap::~ServerMap()
 
        try
        {
-               if(m_map_saving_enabled)
-               {
+               if (m_map_saving_enabled) {
                        // Save only changed parts
                        save(MOD_STATE_WRITE_AT_UNLOAD);
-                       infostream<<"ServerMap: Saved map to "<<m_savedir<<std::endl;
-               }
-               else
-               {
-                       infostream<<"ServerMap: Map not saved"<<std::endl;
+                       infostream << "ServerMap: Saved map to " << m_savedir << std::endl;
+               } else {
+                       infostream << "ServerMap: Map not saved" << std::endl;
                }
        }
        catch(std::exception &e)
@@ -1866,6 +1261,24 @@ s16 ServerMap::getWaterLevel()
        return getMapgenParams()->water_level;
 }
 
+bool ServerMap::saoPositionOverLimit(const v3f &p)
+{
+       return getMapgenParams()->saoPosOverLimit(p);
+}
+
+bool ServerMap::blockpos_over_mapgen_limit(v3s16 p)
+{
+       const s16 mapgen_limit_bp = rangelim(
+               getMapgenParams()->mapgen_limit, 0, MAX_MAP_GENERATION_LIMIT) /
+               MAP_BLOCKSIZE;
+       return p.X < -mapgen_limit_bp ||
+               p.X >  mapgen_limit_bp ||
+               p.Y < -mapgen_limit_bp ||
+               p.Y >  mapgen_limit_bp ||
+               p.Z < -mapgen_limit_bp ||
+               p.Z >  mapgen_limit_bp;
+}
+
 bool ServerMap::initBlockMake(v3s16 blockpos, BlockMakeData *data)
 {
        s16 csize = getMapgenParams()->chunksize;
@@ -1879,9 +1292,9 @@ bool ServerMap::initBlockMake(v3s16 blockpos, BlockMakeData *data)
        v3s16 full_bpmin = bpmin - extra_borders;
        v3s16 full_bpmax = bpmax + extra_borders;
 
-       // Do nothing if not inside limits (+-1 because of neighbors)
-       if (blockpos_over_limit(full_bpmin) ||
-               blockpos_over_limit(full_bpmax))
+       // Do nothing if not inside mapgen limits (+-1 because of neighbors)
+       if (blockpos_over_mapgen_limit(full_bpmin) ||
+                       blockpos_over_mapgen_limit(full_bpmax))
                return false;
 
        data->seed = getSeed();
@@ -1897,7 +1310,7 @@ bool ServerMap::initBlockMake(v3s16 blockpos, BlockMakeData *data)
        for (s16 z = full_bpmin.Z; z <= full_bpmax.Z; z++) {
                v2s16 sectorpos(x, z);
                // Sector metadata is loaded from disk if not already loaded.
-               ServerMapSector *sector = createSector(sectorpos);
+               MapSector *sector = createSector(sectorpos);
                FATAL_ERROR_IF(sector == NULL, "createSector() failed");
 
                for (s16 y = full_bpmin.Y; y <= full_bpmax.Y; y++) {
@@ -1976,10 +1389,8 @@ void ServerMap::finishBlockMake(BlockMakeData *data,
                data->transforming_liquid.pop_front();
        }
 
-       for (std::map<v3s16, MapBlock *>::iterator
-                       it = changed_blocks->begin();
-                       it != changed_blocks->end(); ++it) {
-               MapBlock *block = it->second;
+       for (auto &changed_block : *changed_blocks) {
+               MapBlock *block = changed_block.second;
                if (!block)
                        continue;
                /*
@@ -2013,66 +1424,30 @@ void ServerMap::finishBlockMake(BlockMakeData *data,
        //save(MOD_STATE_WRITE_AT_UNLOAD);
 }
 
-ServerMapSector *ServerMap::createSector(v2s16 p2d)
+MapSector *ServerMap::createSector(v2s16 p2d)
 {
-       DSTACKF("%s: p2d=(%d,%d)",
-                       FUNCTION_NAME,
-                       p2d.X, p2d.Y);
-
        /*
                Check if it exists already in memory
        */
-       ServerMapSector *sector = (ServerMapSector*)getSectorNoGenerateNoEx(p2d);
-       if(sector != NULL)
-               return sector;
-
-       /*
-               Try to load it from disk (with blocks)
-       */
-       //if(loadSectorFull(p2d) == true)
-
-       /*
-               Try to load metadata from disk
-       */
-#if 0
-       if(loadSectorMeta(p2d) == true)
-       {
-               ServerMapSector *sector = (ServerMapSector*)getSectorNoGenerateNoEx(p2d);
-               if(sector == NULL)
-               {
-                       infostream<<"ServerMap::createSector(): loadSectorFull didn't make a sector"<<std::endl;
-                       throw InvalidPositionException("");
-               }
+       MapSector *sector = getSectorNoGenerateNoEx(p2d);
+       if (sector)
                return sector;
-       }
-#endif
 
        /*
-               Do not create over-limit.
-               We are checking for any nodes of the mapblocks of the sector being beyond the limit.
-               A sector is a vertical column of mapblocks, so sectorpos is like a 2D blockpos.
-
-               At the negative limit we are checking for
-                       block minimum nodepos < -mapgenlimit.
-               At the positive limit we are checking for
-                       block maximum nodepos > mapgenlimit.
-
-               Block minimum nodepos = blockpos * mapblocksize.
-               Block maximum nodepos = (blockpos + 1) * mapblocksize - 1.
+               Do not create over max mapgen limit
        */
-       const u16 map_gen_limit = MYMIN(MAX_MAP_GENERATION_LIMIT,
-               g_settings->getU16("map_generation_limit"));
-       if (p2d.X * MAP_BLOCKSIZE < -map_gen_limit
-                       || (p2d.X + 1) * MAP_BLOCKSIZE - 1 > map_gen_limit
-                       || p2d.Y * MAP_BLOCKSIZE < -map_gen_limit
-                       || (p2d.Y + 1) * MAP_BLOCKSIZE - 1 > map_gen_limit)
-               throw InvalidPositionException("createSector(): pos. over limit");
+       const s16 max_limit_bp = MAX_MAP_GENERATION_LIMIT / MAP_BLOCKSIZE;
+       if (p2d.X < -max_limit_bp ||
+                       p2d.X >  max_limit_bp ||
+                       p2d.Y < -max_limit_bp ||
+                       p2d.Y >  max_limit_bp)
+               throw InvalidPositionException("createSector(): pos. over max mapgen limit");
 
        /*
                Generate blank sector
        */
 
-       sector = new ServerMapSector(this, p2d, m_gamedef);
+       sector = new MapSector(this, p2d, m_gamedef);
 
        // Sector position on map in nodes
        //v2s16 nodepos2d = p2d * MAP_BLOCKSIZE;
@@ -2094,12 +1469,6 @@ MapBlock * ServerMap::generateBlock(
                std::map<v3s16, MapBlock*> &modified_blocks
 )
 {
-       DSTACKF("%s: p=(%d,%d,%d)", FUNCTION_NAME, p.X, p.Y, p.Z);
-
-       /*infostream<<"generateBlock(): "
-                       <<"("<<p.X<<","<<p.Y<<","<<p.Z<<")"
-                       <<std::endl;*/
-
        bool enable_mapgen_debug_info = g_settings->getBool("enable_mapgen_debug_info");
 
        TimeTaker timer("generateBlock");
@@ -2203,14 +1572,11 @@ MapBlock * ServerMap::generateBlock(
 
 MapBlock * ServerMap::createBlock(v3s16 p)
 {
-       DSTACKF("%s: p=(%d,%d,%d)",
-                       FUNCTION_NAME, p.X, p.Y, p.Z);
-
        /*
-               Do not create over-limit
+               Do not create over max mapgen limit
        */
-       if (blockpos_over_limit(p))
-               throw InvalidPositionException("createBlock(): pos. over limit");
+       if (blockpos_over_max_limit(p))
+               throw InvalidPositionException("createBlock(): pos. over max mapgen limit");
 
        v2s16 p2d(p.X, p.Z);
        s16 block_y = p.Y;
@@ -2221,35 +1587,20 @@ MapBlock * ServerMap::createBlock(v3s16 p)
                NOTE: On old save formats, this will be slow, as it generates
                      lighting on blocks for them.
        */
-       ServerMapSector *sector;
+       MapSector *sector;
        try {
-               sector = (ServerMapSector*)createSector(p2d);
-               assert(sector->getId() == MAPSECTOR_SERVER);
-       }
-       catch(InvalidPositionException &e)
-       {
+               sector = createSector(p2d);
+       } catch (InvalidPositionException &e) {
                infostream<<"createBlock: createSector() failed"<<std::endl;
                throw e;
        }
-       /*
-               NOTE: This should not be done, or at least the exception
-               should not be passed on as std::exception, because it
-               won't be catched at all.
-       */
-       /*catch(std::exception &e)
-       {
-               infostream<<"createBlock: createSector() failed: "
-                               <<e.what()<<std::endl;
-               throw e;
-       }*/
 
        /*
                Try to get a block from the sector
        */
 
        MapBlock *block = sector->getBlockNoCreateNoEx(block_y);
-       if(block)
-       {
+       if (block) {
                if(block->isDummy())
                        block->unDummify();
                return block;
@@ -2262,13 +1613,9 @@ MapBlock * ServerMap::createBlock(v3s16 p)
 
 MapBlock * ServerMap::emergeBlock(v3s16 p, bool create_blank)
 {
-       DSTACKF("%s: p=(%d,%d,%d), create_blank=%d",
-                       FUNCTION_NAME,
-                       p.X, p.Y, p.Z, create_blank);
-
        {
                MapBlock *block = getBlockNoCreateNoEx(p);
-               if(block && block->isDummy() == false)
+               if (block && !block->isDummy())
                        return block;
        }
 
@@ -2279,39 +1626,12 @@ MapBlock * ServerMap::emergeBlock(v3s16 p, bool create_blank)
        }
 
        if (create_blank) {
-               ServerMapSector *sector = createSector(v2s16(p.X, p.Z));
+               MapSector *sector = createSector(v2s16(p.X, p.Z));
                MapBlock *block = sector->createBlankBlock(p.Y);
 
                return block;
        }
 
-#if 0
-       if(allow_generate)
-       {
-               std::map<v3s16, MapBlock*> modified_blocks;
-               MapBlock *block = generateBlock(p, modified_blocks);
-               if(block)
-               {
-                       MapEditEvent event;
-                       event.type = MEET_OTHER;
-                       event.p = p;
-
-                       // Copy modified_blocks to event
-                       for(std::map<v3s16, MapBlock*>::iterator
-                                       i = modified_blocks.begin();
-                                       i != modified_blocks.end(); ++i)
-                       {
-                               event.modified_blocks.insert(i->first);
-                       }
-
-                       // Queue event
-                       dispatchEvent(&event);
-
-                       return block;
-               }
-       }
-#endif
-
        return NULL;
 }
 
@@ -2324,9 +1644,6 @@ MapBlock *ServerMap::getBlockOrEmerge(v3s16 p3d)
        return block;
 }
 
-void ServerMap::prepareBlock(MapBlock *block) {
-}
-
 // N.B.  This requires no synchronization, since data will not be modified unless
 // the VoxelManipulator being updated belongs to the same thread.
 void ServerMap::updateVManip(v3s16 pos)
@@ -2402,8 +1719,7 @@ bool ServerMap::loadFromFolders() {
 
 void ServerMap::createDirs(std::string path)
 {
-       if(fs::CreateAllDirs(path) == false)
-       {
+       if (!fs::CreateAllDirs(path)) {
                m_dout<<"ServerMap: Failed to create directory "
                                <<"\""<<path<<"\""<<std::endl;
                throw BaseException("ServerMap failed to create directory");
@@ -2433,7 +1749,7 @@ std::string ServerMap::getSectorDir(v2s16 pos, int layout)
        }
 }
 
-v2s16 ServerMap::getSectorPos(std::string dirname)
+v2s16 ServerMap::getSectorPos(const std::string &dirname)
 {
        unsigned int x = 0, y = 0;
        int r;
@@ -2463,7 +1779,7 @@ v2s16 ServerMap::getSectorPos(std::string dirname)
        return pos;
 }
 
-v3s16 ServerMap::getBlockPos(std::string sectordir, std::string blockfile)
+v3s16 ServerMap::getBlockPos(const std::string &sectordir, const std::string &blockfile)
 {
        v2s16 p2d = getSectorPos(sectordir);
 
@@ -2486,8 +1802,7 @@ std::string ServerMap::getBlockFilename(v3s16 p)
 
 void ServerMap::save(ModifiedState save_level)
 {
-       DSTACK(FUNCTION_NAME);
-       if(m_map_saving_enabled == false) {
+       if (!m_map_saving_enabled) {
                warningstream<<"Not saving map, saving disabled."<<std::endl;
                return;
        }
@@ -2504,30 +1819,19 @@ void ServerMap::save(ModifiedState save_level)
        // Profile modified reasons
        Profiler modprofiler;
 
-       u32 sector_meta_count = 0;
        u32 block_count = 0;
        u32 block_count_all = 0; // Number of blocks in memory
 
        // Don't do anything with sqlite unless something is really saved
        bool save_started = false;
 
-       for(std::map<v2s16, MapSector*>::iterator i = m_sectors.begin();
-               i != m_sectors.end(); ++i) {
-               ServerMapSector *sector = (ServerMapSector*)i->second;
-               assert(sector->getId() == MAPSECTOR_SERVER);
-
-               if(sector->differs_from_disk || save_level == MOD_STATE_CLEAN) {
-                       saveSectorMeta(sector);
-                       sector_meta_count++;
-               }
+       for (auto &sector_it : m_sectors) {
+               MapSector *sector = sector_it.second;
 
                MapBlockVect blocks;
                sector->getBlocks(blocks);
 
-               for(MapBlockVect::iterator j = blocks.begin();
-                       j != blocks.end(); ++j) {
-                       MapBlock *block = *j;
-
+               for (MapBlock *block : blocks) {
                        block_count_all++;
 
                        if(block->getModified() >= (u32)save_level) {
@@ -2541,12 +1845,6 @@ void ServerMap::save(ModifiedState save_level)
 
                                saveBlock(block);
                                block_count++;
-
-                               /*infostream<<"ServerMap: Written block ("
-                                               <<block->getPos().X<<","
-                                               <<block->getPos().Y<<","
-                                               <<block->getPos().Z<<")"
-                                               <<std::endl;*/
                        }
                }
        }
@@ -2557,10 +1855,9 @@ void ServerMap::save(ModifiedState save_level)
        /*
                Only print if something happened or saved whole map
        */
-       if(save_level == MOD_STATE_CLEAN || sector_meta_count != 0
+       if(save_level == MOD_STATE_CLEAN
                        || block_count != 0) {
                infostream<<"ServerMap: Written: "
-                               <<sector_meta_count<<" sector metadata files, "
                                <<block_count<<" block files"
                                <<", "<<block_count_all<<" blocks in memory."
                                <<std::endl;
@@ -2581,220 +1878,45 @@ void ServerMap::listAllLoadableBlocks(std::vector<v3s16> &dst)
 
 void ServerMap::listAllLoadedBlocks(std::vector<v3s16> &dst)
 {
-       for(std::map<v2s16, MapSector*>::iterator si = m_sectors.begin();
-               si != m_sectors.end(); ++si)
-       {
-               MapSector *sector = si->second;
+       for (auto &sector_it : m_sectors) {
+               MapSector *sector = sector_it.second;
 
                MapBlockVect blocks;
                sector->getBlocks(blocks);
 
-               for(MapBlockVect::iterator i = blocks.begin();
-                               i != blocks.end(); ++i) {
-                       v3s16 p = (*i)->getPos();
+               for (MapBlock *block : blocks) {
+                       v3s16 p = block->getPos();
                        dst.push_back(p);
                }
        }
 }
 
-void ServerMap::saveSectorMeta(ServerMapSector *sector)
-{
-       DSTACK(FUNCTION_NAME);
-       // Format used for writing
-       u8 version = SER_FMT_VER_HIGHEST_WRITE;
-       // Get destination
-       v2s16 pos = sector->getPos();
-       std::string dir = getSectorDir(pos);
-       createDirs(dir);
-
-       std::string fullpath = dir + DIR_DELIM + "meta";
-       std::ostringstream ss(std::ios_base::binary);
-
-       sector->serialize(ss, version);
-
-       if(!fs::safeWriteToFile(fullpath, ss.str()))
-               throw FileNotGoodException("Cannot write sector metafile");
-
-       sector->differs_from_disk = false;
-}
-
-MapSector* ServerMap::loadSectorMeta(std::string sectordir, bool save_after_load)
-{
-       DSTACK(FUNCTION_NAME);
-       // Get destination
-       v2s16 p2d = getSectorPos(sectordir);
-
-       ServerMapSector *sector = NULL;
-
-       std::string fullpath = sectordir + DIR_DELIM + "meta";
-       std::ifstream is(fullpath.c_str(), std::ios_base::binary);
-       if(is.good() == false)
-       {
-               // If the directory exists anyway, it probably is in some old
-               // format. Just go ahead and create the sector.
-               if(fs::PathExists(sectordir))
-               {
-                       /*infostream<<"ServerMap::loadSectorMeta(): Sector metafile "
-                                       <<fullpath<<" doesn't exist but directory does."
-                                       <<" Continuing with a sector with no metadata."
-                                       <<std::endl;*/
-                       sector = new ServerMapSector(this, p2d, m_gamedef);
-                       m_sectors[p2d] = sector;
-               }
-               else
-               {
-                       throw FileNotGoodException("Cannot open sector metafile");
-               }
-       }
-       else
-       {
-               sector = ServerMapSector::deSerialize
-                               (is, this, p2d, m_sectors, m_gamedef);
-               if(save_after_load)
-                       saveSectorMeta(sector);
-       }
-
-       sector->differs_from_disk = false;
-
-       return sector;
-}
-
-bool ServerMap::loadSectorMeta(v2s16 p2d)
-{
-       DSTACK(FUNCTION_NAME);
-
-       // The directory layout we're going to load from.
-       //  1 - original sectors/xxxxzzzz/
-       //  2 - new sectors2/xxx/zzz/
-       //  If we load from anything but the latest structure, we will
-       //  immediately save to the new one, and remove the old.
-       int loadlayout = 1;
-       std::string sectordir1 = getSectorDir(p2d, 1);
-       std::string sectordir;
-       if(fs::PathExists(sectordir1))
-       {
-               sectordir = sectordir1;
-       }
-       else
-       {
-               loadlayout = 2;
-               sectordir = getSectorDir(p2d, 2);
-       }
-
-       try{
-               loadSectorMeta(sectordir, loadlayout != 2);
-       }
-       catch(InvalidFilenameException &e)
-       {
-               return false;
-       }
-       catch(FileNotGoodException &e)
-       {
-               return false;
-       }
-       catch(std::exception &e)
-       {
-               return false;
-       }
-
-       return true;
-}
-
-#if 0
-bool ServerMap::loadSectorFull(v2s16 p2d)
-{
-       DSTACK(FUNCTION_NAME);
-
-       MapSector *sector = NULL;
-
-       // The directory layout we're going to load from.
-       //  1 - original sectors/xxxxzzzz/
-       //  2 - new sectors2/xxx/zzz/
-       //  If we load from anything but the latest structure, we will
-       //  immediately save to the new one, and remove the old.
-       int loadlayout = 1;
-       std::string sectordir1 = getSectorDir(p2d, 1);
-       std::string sectordir;
-       if(fs::PathExists(sectordir1))
-       {
-               sectordir = sectordir1;
-       }
-       else
-       {
-               loadlayout = 2;
-               sectordir = getSectorDir(p2d, 2);
-       }
-
-       try{
-               sector = loadSectorMeta(sectordir, loadlayout != 2);
-       }
-       catch(InvalidFilenameException &e)
-       {
-               return false;
-       }
-       catch(FileNotGoodException &e)
-       {
-               return false;
-       }
-       catch(std::exception &e)
-       {
-               return false;
-       }
-
-       /*
-               Load blocks
-       */
-       std::vector<fs::DirListNode> list2 = fs::GetDirListing
-                       (sectordir);
-       std::vector<fs::DirListNode>::iterator i2;
-       for(i2=list2.begin(); i2!=list2.end(); i2++)
-       {
-               // We want files
-               if(i2->dir)
-                       continue;
-               try{
-                       loadBlock(sectordir, i2->name, sector, loadlayout != 2);
-               }
-               catch(InvalidFilenameException &e)
-               {
-                       // This catches unknown crap in directory
-               }
-       }
-
-       if(loadlayout != 2)
-       {
-               infostream<<"Sector converted to new layout - deleting "<<
-                       sectordir1<<std::endl;
-               fs::RecursiveDelete(sectordir1);
-       }
-
-       return true;
-}
-#endif
-
-Database *ServerMap::createDatabase(
+MapDatabase *ServerMap::createDatabase(
        const std::string &name,
        const std::string &savedir,
        Settings &conf)
 {
        if (name == "sqlite3")
-               return new Database_SQLite3(savedir);
+               return new MapDatabaseSQLite3(savedir);
        if (name == "dummy")
                return new Database_Dummy();
        #if USE_LEVELDB
-       else if (name == "leveldb")
+       if (name == "leveldb")
                return new Database_LevelDB(savedir);
        #endif
        #if USE_REDIS
-       else if (name == "redis")
+       if (name == "redis")
                return new Database_Redis(conf);
        #endif
        #if USE_POSTGRESQL
-       else if (name == "postgresql")
-               return new Database_PostgreSQL(conf);
+       if (name == "postgresql") {
+               std::string connect_string;
+               conf.getNoEx("pgsql_connection", connect_string);
+               return new MapDatabasePostgreSQL(connect_string);
+       }
        #endif
-       else
-               throw BaseException(std::string("Database backend ") + name + " not supported.");
+
+       throw BaseException(std::string("Database backend ") + name + " not supported.");
 }
 
 void ServerMap::beginSave()
@@ -2812,7 +1934,7 @@ bool ServerMap::saveBlock(MapBlock *block)
        return saveBlock(block, dbase);
 }
 
-bool ServerMap::saveBlock(MapBlock *block, Database *db)
+bool ServerMap::saveBlock(MapBlock *block, MapDatabase *db)
 {
        v3s16 p3d = block->getPos();
 
@@ -2834,8 +1956,7 @@ bool ServerMap::saveBlock(MapBlock *block, Database *db)
        o.write((char*) &version, 1);
        block->serialize(o, version, true);
 
-       std::string data = o.str();
-       bool ret = db->saveBlock(p3d, data);
+       bool ret = db->saveBlock(p3d, o.str());
        if (ret) {
                // We just wrote it to the disk so clear modified flag
                block->resetModified();
@@ -2843,16 +1964,13 @@ bool ServerMap::saveBlock(MapBlock *block, Database *db)
        return ret;
 }
 
-void ServerMap::loadBlock(std::string sectordir, std::string blockfile,
+void ServerMap::loadBlock(const std::string &sectordir, const std::string &blockfile,
                MapSector *sector, bool save_after_load)
 {
-       DSTACK(FUNCTION_NAME);
-
        std::string fullpath = sectordir + DIR_DELIM + blockfile;
        try {
-
                std::ifstream is(fullpath.c_str(), std::ios_base::binary);
-               if(is.good() == false)
+               if (!is.good())
                        throw FileNotGoodException("Cannot open block file");
 
                v3s16 p3d = getBlockPos(sectordir, blockfile);
@@ -2925,8 +2043,6 @@ void ServerMap::loadBlock(std::string sectordir, std::string blockfile,
 
 void ServerMap::loadBlock(std::string *blob, v3s16 p3d, MapSector *sector, bool save_after_load)
 {
-       DSTACK(FUNCTION_NAME);
-
        try {
                std::istringstream is(*blob, std::ios_base::binary);
 
@@ -2937,13 +2053,6 @@ void ServerMap::loadBlock(std::string *blob, v3s16 p3d, MapSector *sector, bool
                        throw SerializationError("ServerMap::loadBlock(): Failed"
                                        " to read MapBlock version");
 
-               /*u32 block_size = MapBlock::serializedLength(version);
-               SharedBuffer<u8> data(block_size);
-               is.read((char*)*data, block_size);*/
-
-               // This will always return a sector because we're the server
-               //MapSector *sector = emergeSector(p2d);
-
                MapBlock *block = NULL;
                bool created_new = false;
                block = sector->getBlockNoCreateNoEx(p3d.Y);
@@ -2995,15 +2104,13 @@ void ServerMap::loadBlock(std::string *blob, v3s16 p3d, MapSector *sector, bool
 
 MapBlock* ServerMap::loadBlock(v3s16 blockpos)
 {
-       DSTACK(FUNCTION_NAME);
-
        bool created_new = (getBlockNoCreateNoEx(blockpos) == NULL);
 
        v2s16 p2d(blockpos.X, blockpos.Z);
 
        std::string ret;
        dbase->loadBlock(blockpos, &ret);
-       if (ret != "") {
+       if (!ret.empty()) {
                loadBlock(&ret, blockpos, createSector(p2d), false);
        } else {
                // Not found in database, try the files
@@ -3013,13 +2120,11 @@ MapBlock* ServerMap::loadBlock(v3s16 blockpos)
                //  2 - new sectors2/xxx/zzz/
                //  If we load from anything but the latest structure, we will
                //  immediately save to the new one, and remove the old.
-               int loadlayout = 1;
                std::string sectordir1 = getSectorDir(p2d, 1);
                std::string sectordir;
                if (fs::PathExists(sectordir1)) {
                        sectordir = sectordir1;
                } else {
-                       loadlayout = 2;
                        sectordir = getSectorDir(p2d, 2);
                }
 
@@ -3028,25 +2133,13 @@ MapBlock* ServerMap::loadBlock(v3s16 blockpos)
                 */
 
                MapSector *sector = getSectorNoGenerateNoEx(p2d);
-               if (sector == NULL) {
-                       try {
-                               sector = loadSectorMeta(sectordir, loadlayout != 2);
-                       } catch(InvalidFilenameException &e) {
-                               return NULL;
-                       } catch(FileNotGoodException &e) {
-                               return NULL;
-                       } catch(std::exception &e) {
-                               return NULL;
-                       }
-               }
-
 
                /*
                Make sure file exists
                 */
 
                std::string blockfilename = getBlockFilename(blockpos);
-               if (fs::PathExists(sectordir + DIR_DELIM + blockfilename) == false)
+               if (!fs::PathExists(sectordir + DIR_DELIM + blockfilename))
                        return NULL;
 
                /*
@@ -3054,6 +2147,7 @@ MapBlock* ServerMap::loadBlock(v3s16 blockpos)
                 */
                loadBlock(sectordir, blockfilename, sector, true);
        }
+
        MapBlock *block = getBlockNoCreateNoEx(blockpos);
        if (created_new && (block != NULL)) {
                std::map<v3s16, MapBlock*> modified_blocks;
@@ -3095,15 +2189,19 @@ void ServerMap::PrintInfo(std::ostream &out)
        out<<"ServerMap: ";
 }
 
-MMVManip::MMVManip(Map *map):
-               VoxelManipulator(),
-               m_is_dirty(false),
-               m_create_area(false),
-               m_map(map)
+bool ServerMap::repairBlockLight(v3s16 blockpos,
+       std::map<v3s16, MapBlock *> *modified_blocks)
 {
+       MapBlock *block = emergeBlock(blockpos, false);
+       if (!block || !block->isGenerated())
+               return false;
+       voxalgo::repair_block_light(this, block, modified_blocks);
+       return true;
 }
 
-MMVManip::~MMVManip()
+MMVManip::MMVManip(Map *map):
+               VoxelManipulator(),
+               m_map(map)
 {
 }
 
@@ -3161,7 +2259,7 @@ void MMVManip::initialEmerge(v3s16 blockpos_min, v3s16 blockpos_max,
                if(block_data_inexistent)
                {
 
-                       if (load_if_inexistent) {
+                       if (load_if_inexistent && !blockpos_over_max_limit(p)) {
                                ServerMap *svrmap = (ServerMap *)m_map;
                                block = svrmap->emergeBlock(p, false);
                                if (block == NULL)
@@ -3204,18 +2302,16 @@ void MMVManip::blitBackAll(std::map<v3s16, MapBlock*> *modified_blocks,
        /*
                Copy data of all blocks
        */
-       for(std::map<v3s16, u8>::iterator
-                       i = m_loaded_blocks.begin();
-                       i != m_loaded_blocks.end(); ++i)
-       {
-               v3s16 p = i->first;
+       for (auto &loaded_block : m_loaded_blocks) {
+               v3s16 p = loaded_block.first;
                MapBlock *block = m_map->getBlockNoCreateNoEx(p);
-               bool existed = !(i->second & VMANIP_BLOCK_DATA_INEXIST);
-               if ((existed == false) || (block == NULL) ||
-                       (overwrite_generated == false && block->isGenerated() == true))
+               bool existed = !(loaded_block.second & VMANIP_BLOCK_DATA_INEXIST);
+               if (!existed || (block == NULL) ||
+                       (!overwrite_generated && block->isGenerated()))
                        continue;
 
                block->copyFrom(*this);
+               block->raiseModified(MOD_STATE_WRITE_NEEDED, MOD_REASON_VMANIP);
 
                if(modified_blocks)
                        (*modified_blocks)[p] = block;