Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
Remove emerge and speedup addArea by using memcopy instead of one by …
…one assignment
  • Loading branch information
sapier authored and sapier committed Jun 22, 2014
1 parent d5de0cb commit 8ad8376
Show file tree
Hide file tree
Showing 4 changed files with 83 additions and 112 deletions.
12 changes: 6 additions & 6 deletions src/map.cpp
Expand Up @@ -3551,12 +3551,12 @@ void MapVoxelManipulator::emerge(VoxelArea a, s32 caller_id)
flags |= VMANIP_BLOCK_DATA_INEXIST;

VoxelArea a(p*MAP_BLOCKSIZE, (p+1)*MAP_BLOCKSIZE-v3s16(1,1,1));
// Fill with VOXELFLAG_INEXISTENT
// Fill with VOXELFLAG_NO_DATA
for(s32 z=a.MinEdge.Z; z<=a.MaxEdge.Z; z++)
for(s32 y=a.MinEdge.Y; y<=a.MaxEdge.Y; y++)
{
s32 i = m_area.index(a.MinEdge.X,y,z);
memset(&m_flags[i], VOXELFLAG_INEXISTENT, MAP_BLOCKSIZE);
memset(&m_flags[i], VOXELFLAG_NO_DATA, MAP_BLOCKSIZE);
}
}
/*else if (block->getNode(0, 0, 0).getContent() == CONTENT_IGNORE)
Expand Down Expand Up @@ -3601,7 +3601,7 @@ void MapVoxelManipulator::blitBack
v3s16 p(x,y,z);

u8 f = m_flags[m_area.index(p)];
if(f & (VOXELFLAG_NOT_LOADED|VOXELFLAG_INEXISTENT))
if(f & (VOXELFLAG_NO_DATA))
continue;

MapNode &n = m_data[m_area.index(p)];
Expand Down Expand Up @@ -3655,7 +3655,7 @@ ManualMapVoxelManipulator::~ManualMapVoxelManipulator()
void ManualMapVoxelManipulator::emerge(VoxelArea a, s32 caller_id)
{
// Just create the area so that it can be pointed to
VoxelManipulator::emerge(a, caller_id);
VoxelManipulator::addArea(a);
}

void ManualMapVoxelManipulator::initialEmerge(v3s16 blockpos_min,
Expand Down Expand Up @@ -3726,12 +3726,12 @@ void ManualMapVoxelManipulator::initialEmerge(v3s16 blockpos_min,
Mark area inexistent
*/
VoxelArea a(p*MAP_BLOCKSIZE, (p+1)*MAP_BLOCKSIZE-v3s16(1,1,1));
// Fill with VOXELFLAG_INEXISTENT
// Fill with VOXELFLAG_NO_DATA
for(s32 z=a.MinEdge.Z; z<=a.MaxEdge.Z; z++)
for(s32 y=a.MinEdge.Y; y<=a.MaxEdge.Y; y++)
{
s32 i = m_area.index(a.MinEdge.X,y,z);
memset(&m_flags[i], VOXELFLAG_INEXISTENT, MAP_BLOCKSIZE);
memset(&m_flags[i], VOXELFLAG_NO_DATA, MAP_BLOCKSIZE);
}
}
}
Expand Down
111 changes: 53 additions & 58 deletions src/voxel.cpp
Expand Up @@ -71,7 +71,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)
Expand All @@ -88,10 +88,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';
Expand Down Expand Up @@ -149,11 +147,11 @@ void VoxelManipulator::addArea(VoxelArea area)
// Cancel if requested area has zero volume
if(area.getExtent() == v3s16(0,0,0))
return;

// Cancel if m_area already contains the requested area
if(m_area.contains(area))
return;

TimeTaker timer("addArea", &addarea_time);

// Calculate new area
Expand Down Expand Up @@ -186,28 +184,26 @@ void VoxelManipulator::addArea(VoxelArea area)
assert(new_data);
u8 *new_flags = new u8[new_size];
assert(new_flags);
memset(new_flags, VOXELFLAG_NOT_LOADED, new_size);
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++)
{
unsigned int old_index = m_area.index(x,y,z);
// If loaded, copy data and flags
if((m_flags[old_index] & VOXELFLAG_NOT_LOADED) == false)
{
unsigned int new_index = new_area.index(x,y,z);
new_data[new_index] = m_data[old_index];
new_flags[new_index] = m_flags[old_index];
}
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;

Expand All @@ -216,7 +212,7 @@ void VoxelManipulator::addArea(VoxelArea area)

m_data = new_data;
m_flags = new_flags;

if(old_data)
delete[] old_data;
if(old_flags)
Expand All @@ -225,7 +221,7 @@ void VoxelManipulator::addArea(VoxelArea area)
//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++)
Expand All @@ -238,7 +234,7 @@ void VoxelManipulator::copyFrom(MapNode *src, VoxelArea src_area,
}
}

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++)
Expand All @@ -252,7 +248,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));
}
}

Expand Down Expand Up @@ -314,22 +309,22 @@ 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)));

addArea(VoxelArea(p - v3s16(1,1,1), p + v3s16(1,1,1)));

// 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)
Expand All @@ -347,9 +342,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
Expand Down Expand Up @@ -391,7 +386,7 @@ void VoxelManipulator::unspreadLight(enum LightBank bank,
{
if(from_nodes.size() == 0)
return;

for(std::map<v3s16, u8>::iterator j = from_nodes.begin();
j != from_nodes.end(); ++j)
{
Expand Down Expand Up @@ -430,37 +425,37 @@ 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();

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)
Expand All @@ -480,7 +475,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
Expand All @@ -502,7 +497,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);
}
Expand All @@ -520,11 +515,11 @@ 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)));
addArea(VoxelArea(p - v3s16(1,1,1), p + v3s16(1,1,1)));

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];
Expand All @@ -537,16 +532,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)
Expand Down Expand Up @@ -583,7 +578,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();
Expand Down Expand Up @@ -616,19 +611,19 @@ void VoxelManipulator::spreadLight(enum LightBank bank,

if(from_nodes.size() == 0)
return;

std::set<v3s16> lighted_nodes;

for(std::set<v3s16>::iterator j = from_nodes.begin();
j != from_nodes.end(); ++j)
{
v3s16 pos = *j;

emerge(VoxelArea(pos - v3s16(1,1,1), pos + v3s16(1,1,1)));
addArea(VoxelArea(pos - v3s16(1,1,1), pos + v3s16(1,1,1)));

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];
Expand All @@ -641,18 +636,18 @@ 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)
Expand Down Expand Up @@ -685,7 +680,7 @@ void VoxelManipulator::spreadLight(enum LightBank bank,
<<blockchangecount<<" times"
<<" for "<<from_nodes.size()<<" nodes"
<<std::endl;*/

if(lighted_nodes.size() > 0)
spreadLight(bank, lighted_nodes, nodemgr);
}
Expand Down

0 comments on commit 8ad8376

Please sign in to comment.