oss/luanti
oss/luanti
1
0
Fork 0
mirror of https://github.com/luanti-org/luanti.git synced 2025-06-27 16:36:03 +00:00
Code Wiki Activity

Fix m_nearest_unsent_d calculation when changed blocks arrive

Browse source 
Uses the nearest cube surface instead of euclidean distance to correctly reset the distance to start scanning from.
This commit is contained in:
Lars 2025-04-12 09:38:03 -07:00 • committed by lhofhansl
parent 99150fadd1
commit 2a3427a39c
1 changed files with 10 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

11
src/server/clientiface.cpp
View file

@ -429,7 +429,16 @@ void RemoteClient::SetBlockNotSent(v3s16 p)
// remove the block from sending and sent sets, // remove the block from sending and sent sets,
// and reset the scan loop if found // and reset the scan loop if found
if (m_blocks_sending.erase(p) + m_blocks_sent.erase(p) > 0) { if (m_blocks_sending.erase(p) + m_blocks_sent.erase(p) > 0) {
m_nearest_unsent_d = std::min(m_nearest_unsent_d, m_last_center.getDistanceFrom(p)); // Note that we do NOT use the euclidean distance here.
// getNextBlocks builds successive cube-surfaces in the send loop.
// This resets the distance to the maximum cube size that
// still guarantees that this block will be scanned again right away.
//
// Using m_last_center is OK, as a change in center
// will reset m_nearest_unsent_d to 0 anyway (see getNextBlocks).
p -= m_last_center;
s16 this_d = std::max({std::abs(p.X), std::abs(p.Y), std::abs(p.Z)});
m_nearest_unsent_d = std::min(m_nearest_unsent_d, this_d);
} }
} }