]> git.lizzy.rs Git - dragonfireclient.git/blobdiff - src/voxel.cpp
Optimise MapBlockMesh related functions
[dragonfireclient.git] / src / voxel.cpp
index c55f3f5392dd0716ba96096412932aa91e2bf273..a5e0b09fe7517ce334050a29fa7259329021289a 100644 (file)
@@ -22,6 +22,7 @@ with this program; if not, write to the Free Software Foundation, Inc.,
 #include "gettime.h"
 #include "nodedef.h"
 #include "util/timetaker.h"
+#include <string.h>  // memcpy, memset
 
 /*
        Debug stuff
@@ -45,21 +46,15 @@ VoxelManipulator::VoxelManipulator():
 VoxelManipulator::~VoxelManipulator()
 {
        clear();
-       if(m_data)
-               delete[] m_data;
-       if(m_flags)
-               delete[] m_flags;
 }
 
 void VoxelManipulator::clear()
 {
        // Reset area to volume=0
        m_area = VoxelArea();
-       if(m_data)
-               delete[] m_data;
+       delete[] m_data;
        m_data = NULL;
-       if(m_flags)
-               delete[] m_flags;
+       delete[] m_flags;
        m_flags = NULL;
 }
 
@@ -70,7 +65,7 @@ void VoxelManipulator::print(std::ostream &o, INodeDefManager *ndef,
        v3s16 of = m_area.MinEdge;
        o<<"size: "<<em.X<<"x"<<em.Y<<"x"<<em.Z
         <<" offset: ("<<of.X<<","<<of.Y<<","<<of.Z<<")"<<std::endl;
-       
+
        for(s32 y=m_area.MaxEdge.Y; y>=m_area.MinEdge.Y; y--)
        {
                if(em.X >= 3 && em.Y >= 3)
@@ -87,10 +82,8 @@ void VoxelManipulator::print(std::ostream &o, INodeDefManager *ndef,
                        {
                                u8 f = m_flags[m_area.index(x,y,z)];
                                char c;
-                               if(f & VOXELFLAG_NOT_LOADED)
+                               if(f & VOXELFLAG_NO_DATA)
                                        c = 'N';
-                               else if(f & VOXELFLAG_INEXISTENT)
-                                       c = 'I';
                                else
                                {
                                        c = 'X';
@@ -143,22 +136,22 @@ void VoxelManipulator::print(std::ostream &o, INodeDefManager *ndef,
        }
 }
 
-void VoxelManipulator::addArea(VoxelArea area)
+void VoxelManipulator::addArea(const VoxelArea &area)
 {
        // Cancel if requested area has zero volume
-       if(area.getExtent() == v3s16(0,0,0))
+       if (area.hasEmptyExtent())
                return;
-       
+
        // Cancel if m_area already contains the requested area
        if(m_area.contains(area))
                return;
-       
+
        TimeTaker timer("addArea", &addarea_time);
 
        // Calculate new area
        VoxelArea new_area;
        // New area is the requested area if m_area has zero volume
-       if(m_area.getExtent() == v3s16(0,0,0))
+       if(m_area.hasEmptyExtent())
        {
                new_area = area;
        }
@@ -180,32 +173,31 @@ void VoxelManipulator::addArea(VoxelArea area)
        dstream<<", new_size="<<new_size;
        dstream<<std::endl;*/
 
-       // Allocate and clear new data
+       // Allocate new data and clear flags
        MapNode *new_data = new MapNode[new_size];
+       assert(new_data);
        u8 *new_flags = new u8[new_size];
-       for(s32 i=0; i<new_size; i++)
-       {
-               new_flags[i] = VOXELFLAG_NOT_LOADED;
-       }
-       
+       assert(new_flags);
+       memset(new_flags, VOXELFLAG_NO_DATA, new_size);
+
        // Copy old data
-       
+       s32 old_x_width = m_area.MaxEdge.X - m_area.MinEdge.X + 1;
        for(s32 z=m_area.MinEdge.Z; z<=m_area.MaxEdge.Z; z++)
        for(s32 y=m_area.MinEdge.Y; y<=m_area.MaxEdge.Y; y++)
-       for(s32 x=m_area.MinEdge.X; x<=m_area.MaxEdge.X; x++)
        {
-               // If loaded, copy data and flags
-               if((m_flags[m_area.index(x,y,z)] & VOXELFLAG_NOT_LOADED) == false)
-               {
-                       new_data[new_area.index(x,y,z)] = m_data[m_area.index(x,y,z)];
-                       new_flags[new_area.index(x,y,z)] = m_flags[m_area.index(x,y,z)];
-               }
+               unsigned int old_index = m_area.index(m_area.MinEdge.X,y,z);
+               unsigned int new_index = new_area.index(m_area.MinEdge.X,y,z);
+
+               memcpy(&new_data[new_index], &m_data[old_index],
+                               old_x_width * sizeof(MapNode));
+               memcpy(&new_flags[new_index], &m_flags[old_index],
+                               old_x_width * sizeof(u8));
        }
 
        // Replace area, data and flags
-       
+
        m_area = new_area;
-       
+
        MapNode *old_data = m_data;
        u8 *old_flags = m_flags;
 
@@ -214,29 +206,62 @@ void VoxelManipulator::addArea(VoxelArea area)
 
        m_data = new_data;
        m_flags = new_flags;
-       
-       if(old_data)
-               delete[] old_data;
-       if(old_flags)
-               delete[] old_flags;
+
+       delete[] old_data;
+       delete[] old_flags;
 
        //dstream<<"addArea done"<<std::endl;
 }
 
-void VoxelManipulator::copyFrom(MapNode *src, VoxelArea src_area,
+void VoxelManipulator::copyFrom(MapNode *src, const VoxelArea& src_area,
                v3s16 from_pos, v3s16 to_pos, v3s16 size)
 {
-       for(s16 z=0; z<size.Z; z++)
-       for(s16 y=0; y<size.Y; y++)
-       {
-               s32 i_src = src_area.index(from_pos.X, from_pos.Y+y, from_pos.Z+z);
-               s32 i_local = m_area.index(to_pos.X, to_pos.Y+y, to_pos.Z+z);
-               memcpy(&m_data[i_local], &src[i_src], size.X*sizeof(MapNode));
-               memset(&m_flags[i_local], 0, size.X);
+       /* The reason for this optimised code is that we're a member function
+        * and the data type/layout of m_data is know to us: it's stored as
+        * [z*h*w + y*h + x]. Therefore we can take the calls to m_area index
+        * (which performs the preceding mapping/indexing of m_data) out of the
+        * inner loop and calculate the next index as we're iterating to gain
+        * performance.
+        *
+        * src_step and dest_step is the amount required to be added to our index
+        * every time y increments. Because the destination area may be larger
+        * than the source area we need one additional variable (otherwise we could
+        * just continue adding dest_step as is done for the source data): dest_mod.
+        * dest_mod is the difference in size between a "row" in the source data
+        * and a "row" in the destination data (I am using the term row loosely
+        * and for illustrative purposes). E.g.
+        *
+        * src       <-------------------->|'''''' dest mod ''''''''
+        * dest      <--------------------------------------------->
+        *
+        * dest_mod (it's essentially a modulus) is added to the destination index
+        * after every full iteration of the y span.
+        *
+        * This method falls under the category "linear array and incrementing
+        * index".
+        */
+
+       s32 src_step = src_area.getExtent().X;
+       s32 dest_step = m_area.getExtent().X;
+       s32 dest_mod = m_area.index(to_pos.X, to_pos.Y, to_pos.Z + 1)
+                       - m_area.index(to_pos.X, to_pos.Y, to_pos.Z)
+                       - dest_step * size.Y;
+
+       s32 i_src = src_area.index(from_pos.X, from_pos.Y, from_pos.Z);
+       s32 i_local = m_area.index(to_pos.X, to_pos.Y, to_pos.Z);
+
+       for (s16 z = 0; z < size.Z; z++) {
+               for (s16 y = 0; y < size.Y; y++) {
+                       memcpy(&m_data[i_local], &src[i_src], size.X * sizeof(*m_data));
+                       memset(&m_flags[i_local], 0, size.X);
+                       i_src += src_step;
+                       i_local += dest_step;
+               }
+               i_local += dest_mod;
        }
 }
 
-void VoxelManipulator::copyTo(MapNode *dst, VoxelArea dst_area,
+void VoxelManipulator::copyTo(MapNode *dst, const VoxelArea& dst_area,
                v3s16 dst_pos, v3s16 from_pos, v3s16 size)
 {
        for(s16 z=0; z<size.Z; z++)
@@ -250,7 +275,6 @@ void VoxelManipulator::copyTo(MapNode *dst, VoxelArea dst_area,
                        i_dst++;
                        i_local++;
                }
-               //memcpy(&dst[i_dst], &m_data[i_local], size.X*sizeof(MapNode));
        }
 }
 
@@ -264,7 +288,7 @@ void VoxelManipulator::clearFlag(u8 flags)
        // 0-1ms on moderate area
        TimeTaker timer("clearFlag", &clearflag_time);
 
-       v3s16 s = m_area.getExtent();
+       //v3s16 s = m_area.getExtent();
 
        /*dstream<<"clearFlag clearing area of size "
                        <<""<<s.X<<"x"<<s.Y<<"x"<<s.Z<<""
@@ -302,7 +326,7 @@ void VoxelManipulator::clearFlag(u8 flags)
 }
 
 void VoxelManipulator::unspreadLight(enum LightBank bank, v3s16 p, u8 oldlight,
-               core::map<v3s16, bool> & light_sources, INodeDefManager *nodemgr)
+               std::set<v3s16> & light_sources, INodeDefManager *nodemgr)
 {
        v3s16 dirs[6] = {
                v3s16(0,0,1), // back
@@ -312,22 +336,23 @@ void VoxelManipulator::unspreadLight(enum LightBank bank, v3s16 p, u8 oldlight,
                v3s16(0,-1,0), // bottom
                v3s16(-1,0,0), // left
        };
-       
-       emerge(VoxelArea(p - v3s16(1,1,1), p + v3s16(1,1,1)));
+
+       VoxelArea voxel_area(p - v3s16(1,1,1), p + v3s16(1,1,1));
+       addArea(voxel_area);
 
        // Loop through 6 neighbors
        for(u16 i=0; i<6; i++)
        {
                // Get the position of the neighbor node
                v3s16 n2pos = p + dirs[i];
-               
+
                u32 n2i = m_area.index(n2pos);
 
-               if(m_flags[n2i] & VOXELFLAG_INEXISTENT)
+               if(m_flags[n2i] & VOXELFLAG_NO_DATA)
                        continue;
 
                MapNode &n2 = m_data[n2i];
-               
+
                /*
                        If the neighbor is dimmer than what was specified
                        as oldlight (the light of the previous node)
@@ -345,9 +370,9 @@ void VoxelManipulator::unspreadLight(enum LightBank bank, v3s16 p, u8 oldlight,
                                */
 
                                n2.setLight(bank, 0, nodemgr);
-                               
+
                                unspreadLight(bank, n2pos, light2, light_sources, nodemgr);
-                               
+
                                /*
                                        Remove from light_sources if it is there
                                        NOTE: This doesn't happen nearly at all
@@ -360,7 +385,7 @@ void VoxelManipulator::unspreadLight(enum LightBank bank, v3s16 p, u8 oldlight,
                        }
                }
                else{
-                       light_sources.insert(n2pos, true);
+                       light_sources.insert(n2pos);
                }
        }
 }
@@ -384,24 +409,16 @@ void VoxelManipulator::unspreadLight(enum LightBank bank, v3s16 p, u8 oldlight,
        values of from_nodes are lighting values.
 */
 void VoxelManipulator::unspreadLight(enum LightBank bank,
-               core::map<v3s16, u8> & from_nodes,
-               core::map<v3s16, bool> & light_sources, INodeDefManager *nodemgr)
+               std::map<v3s16, u8> & from_nodes,
+               std::set<v3s16> & light_sources, INodeDefManager *nodemgr)
 {
-       if(from_nodes.size() == 0)
+       if(from_nodes.empty())
                return;
-       
-       core::map<v3s16, u8>::Iterator j;
-       j = from_nodes.getIterator();
 
-       for(; j.atEnd() == false; j++)
+       for(std::map<v3s16, u8>::iterator j = from_nodes.begin();
+               j != from_nodes.end(); ++j)
        {
-               v3s16 pos = j.getNode()->getKey();
-               
-               //MapNode &n = m_data[m_area.index(pos)];
-               
-               u8 oldlight = j.getNode()->getValue();
-
-               unspreadLight(bank, pos, oldlight, light_sources, nodemgr);
+               unspreadLight(bank, j->first, j->second, light_sources, nodemgr);
        }
 }
 #endif
@@ -436,10 +453,10 @@ void VoxelManipulator::unspreadLight(enum LightBank bank,
                v3s16(0,-1,0), // bottom
                v3s16(-1,0,0), // left
        };
-       
+
        if(from_nodes.size() == 0)
                return;
-       
+
        core::map<v3s16, u8> unlighted_nodes;
        core::map<v3s16, u8>::Iterator j;
        j = from_nodes.getIterator();
@@ -447,26 +464,26 @@ void VoxelManipulator::unspreadLight(enum LightBank bank,
        for(; j.atEnd() == false; j++)
        {
                v3s16 pos = j.getNode()->getKey();
-               
-               emerge(VoxelArea(pos - v3s16(1,1,1), pos + v3s16(1,1,1)));
+
+               addArea(VoxelArea(pos - v3s16(1,1,1), pos + v3s16(1,1,1)));
 
                //MapNode &n = m_data[m_area.index(pos)];
-               
+
                u8 oldlight = j.getNode()->getValue();
-               
+
                // Loop through 6 neighbors
                for(u16 i=0; i<6; i++)
                {
                        // Get the position of the neighbor node
                        v3s16 n2pos = pos + dirs[i];
-                       
+
                        u32 n2i = m_area.index(n2pos);
 
-                       if(m_flags[n2i] & VOXELFLAG_INEXISTENT)
+                       if(m_flags[n2i] & VOXELFLAG_NO_DATA)
                                continue;
 
                        MapNode &n2 = m_data[n2i];
-                       
+
                        /*
                                If the neighbor is dimmer than what was specified
                                as oldlight (the light of the previous node)
@@ -486,7 +503,7 @@ void VoxelManipulator::unspreadLight(enum LightBank bank,
                                        n2.setLight(bank, 0);
 
                                        unlighted_nodes.insert(n2pos, current_light);
-                                       
+
                                        /*
                                                Remove from light_sources if it is there
                                                NOTE: This doesn't happen nearly at all
@@ -508,7 +525,7 @@ void VoxelManipulator::unspreadLight(enum LightBank bank,
                        <<blockchangecount<<" times"
                        <<" for "<<from_nodes.size()<<" nodes"
                        <<std::endl;*/
-       
+
        if(unlighted_nodes.size() > 0)
                unspreadLight(bank, unlighted_nodes, light_sources);
 }
@@ -526,11 +543,12 @@ void VoxelManipulator::spreadLight(enum LightBank bank, v3s16 p,
                v3s16(-1,0,0), // left
        };
 
-       emerge(VoxelArea(p - v3s16(1,1,1), p + v3s16(1,1,1)));
+       VoxelArea voxel_area(p - v3s16(1,1,1), p + v3s16(1,1,1));
+       addArea(voxel_area);
 
        u32 i = m_area.index(p);
-       
-       if(m_flags[i] & VOXELFLAG_INEXISTENT)
+
+       if(m_flags[i] & VOXELFLAG_NO_DATA)
                return;
 
        MapNode &n = m_data[i];
@@ -543,16 +561,16 @@ void VoxelManipulator::spreadLight(enum LightBank bank, v3s16 p,
        {
                // Get the position of the neighbor node
                v3s16 n2pos = p + dirs[i];
-               
+
                u32 n2i = m_area.index(n2pos);
 
-               if(m_flags[n2i] & VOXELFLAG_INEXISTENT)
+               if(m_flags[n2i] & VOXELFLAG_NO_DATA)
                        continue;
 
                MapNode &n2 = m_data[n2i];
 
                u8 light2 = n2.getLight(bank, nodemgr);
-               
+
                /*
                        If the neighbor is brighter than the current node,
                        add to list (it will light up this node on its turn)
@@ -589,7 +607,7 @@ void VoxelManipulator::spreadLight(enum LightBank bank,
 {
        if(from_nodes.size() == 0)
                return;
-       
+
        core::map<v3s16, bool> lighted_nodes;
        core::map<v3s16, bool>::Iterator j;
        j = from_nodes.getIterator();
@@ -603,13 +621,15 @@ void VoxelManipulator::spreadLight(enum LightBank bank,
 }
 #endif
 
+const MapNode VoxelManipulator::ContentIgnoreNode = MapNode(CONTENT_IGNORE);
+
 #if 1
 /*
        Lights neighbors of from_nodes, collects all them and then
        goes on recursively.
 */
 void VoxelManipulator::spreadLight(enum LightBank bank,
-               core::map<v3s16, bool> & from_nodes, INodeDefManager *nodemgr)
+               std::set<v3s16> & from_nodes, INodeDefManager *nodemgr)
 {
        const v3s16 dirs[6] = {
                v3s16(0,0,1), // back
@@ -620,22 +640,22 @@ void VoxelManipulator::spreadLight(enum LightBank bank,
                v3s16(-1,0,0), // left
        };
 
-       if(from_nodes.size() == 0)
+       if(from_nodes.empty())
                return;
-       
-       core::map<v3s16, bool> lighted_nodes;
-       core::map<v3s16, bool>::Iterator j;
-       j = from_nodes.getIterator();
 
-       for(; j.atEnd() == false; j++)
+       std::set<v3s16> lighted_nodes;
+
+       for(std::set<v3s16>::iterator j = from_nodes.begin();
+               j != from_nodes.end(); ++j)
        {
-               v3s16 pos = j.getNode()->getKey();
+               v3s16 pos = *j;
 
-               emerge(VoxelArea(pos - v3s16(1,1,1), pos + v3s16(1,1,1)));
+               VoxelArea voxel_area(pos - v3s16(1,1,1), pos + v3s16(1,1,1));
+               addArea(voxel_area);
 
                u32 i = m_area.index(pos);
-               
-               if(m_flags[i] & VOXELFLAG_INEXISTENT)
+
+               if(m_flags[i] & VOXELFLAG_NO_DATA)
                        continue;
 
                MapNode &n = m_data[i];
@@ -648,25 +668,25 @@ void VoxelManipulator::spreadLight(enum LightBank bank,
                {
                        // Get the position of the neighbor node
                        v3s16 n2pos = pos + dirs[i];
-                       
+
                        try
                        {
                                u32 n2i = m_area.index(n2pos);
 
-                               if(m_flags[n2i] & VOXELFLAG_INEXISTENT)
+                               if(m_flags[n2i] & VOXELFLAG_NO_DATA)
                                        continue;
 
                                MapNode &n2 = m_data[n2i];
 
                                u8 light2 = n2.getLight(bank, nodemgr);
-                               
+
                                /*
                                        If the neighbor is brighter than the current node,
                                        add to list (it will light up this node on its turn)
                                */
                                if(light2 > undiminish_light(oldlight))
                                {
-                                       lighted_nodes.insert(n2pos, true);
+                                       lighted_nodes.insert(n2pos);
                                }
                                /*
                                        If the neighbor is dimmer than how much light this node
@@ -677,7 +697,7 @@ void VoxelManipulator::spreadLight(enum LightBank bank,
                                        if(nodemgr->get(n2).light_propagates)
                                        {
                                                n2.setLight(bank, newlight, nodemgr);
-                                               lighted_nodes.insert(n2pos, true);
+                                               lighted_nodes.insert(n2pos);
                                        }
                                }
                        }
@@ -692,8 +712,8 @@ void VoxelManipulator::spreadLight(enum LightBank bank,
                        <<blockchangecount<<" times"
                        <<" for "<<from_nodes.size()<<" nodes"
                        <<std::endl;*/
-       
-       if(lighted_nodes.size() > 0)
+
+       if(!lighted_nodes.empty())
                spreadLight(bank, lighted_nodes, nodemgr);
 }
 #endif