summaryrefslogtreecommitdiffstats
path: root/source/ChunkSender.cpp
diff options
context:
space:
mode:
authormadmaxoft@gmail.com <madmaxoft@gmail.com@0a769ca7-a7f5-676a-18bf-c427514a06d6>2012-03-07 21:06:18 +0100
committermadmaxoft@gmail.com <madmaxoft@gmail.com@0a769ca7-a7f5-676a-18bf-c427514a06d6>2012-03-07 21:06:18 +0100
commit4544d5d3b9fe9c1053f5eaea026b651291712a21 (patch)
tree3854a0eb1e14c0f4b1101985ae0dae12bf914ba2 /source/ChunkSender.cpp
parentWSSCompact: fixed switched version values (!); removed unused methods (diff)
downloadcuberite-4544d5d3b9fe9c1053f5eaea026b651291712a21.tar
cuberite-4544d5d3b9fe9c1053f5eaea026b651291712a21.tar.gz
cuberite-4544d5d3b9fe9c1053f5eaea026b651291712a21.tar.bz2
cuberite-4544d5d3b9fe9c1053f5eaea026b651291712a21.tar.lz
cuberite-4544d5d3b9fe9c1053f5eaea026b651291712a21.tar.xz
cuberite-4544d5d3b9fe9c1053f5eaea026b651291712a21.tar.zst
cuberite-4544d5d3b9fe9c1053f5eaea026b651291712a21.zip
Diffstat (limited to '')
-rw-r--r--source/ChunkSender.cpp30
1 files changed, 18 insertions, 12 deletions
diff --git a/source/ChunkSender.cpp b/source/ChunkSender.cpp
index 3a664962d..f3bc439d1 100644
--- a/source/ChunkSender.cpp
+++ b/source/ChunkSender.cpp
@@ -31,7 +31,7 @@ cChunkSender::cChunkSender(void) :
cChunkSender::~cChunkSender()
{
mShouldTerminate = true;
- m_Event.Set();
+ m_evtQueue.Set();
}
@@ -55,7 +55,7 @@ void cChunkSender::ChunkReady(int a_ChunkX, int a_ChunkY, int a_ChunkZ)
cCSLock Lock(m_CS);
m_ChunksReady.push_back(cChunkCoords(a_ChunkX, a_ChunkY, a_ChunkZ));
}
- m_Event.Set();
+ m_evtQueue.Set();
}
@@ -69,7 +69,7 @@ void cChunkSender::QueueSendChunkTo(int a_ChunkX, int a_ChunkY, int a_ChunkZ, cC
cCSLock Lock(m_CS);
m_SendChunks.push_back(sSendChunk(a_ChunkX, a_ChunkY, a_ChunkZ, a_Client));
}
- m_Event.Set();
+ m_evtQueue.Set();
}
@@ -78,16 +78,20 @@ void cChunkSender::QueueSendChunkTo(int a_ChunkX, int a_ChunkY, int a_ChunkZ, cC
void cChunkSender::RemoveClient(cClientHandle * a_Client)
{
- cCSLock Lock(m_CS);
- for (sSendChunkList::iterator itr = m_SendChunks.begin(); itr != m_SendChunks.end();)
{
- if (itr->m_Client == a_Client)
+ cCSLock Lock(m_CS);
+ for (sSendChunkList::iterator itr = m_SendChunks.begin(); itr != m_SendChunks.end();)
{
- itr = m_SendChunks.erase(itr);
- continue;
- }
- ++itr;
- } // for itr - m_SendChunks[]
+ if (itr->m_Client == a_Client)
+ {
+ itr = m_SendChunks.erase(itr);
+ continue;
+ }
+ ++itr;
+ } // for itr - m_SendChunks[]
+ }
+ m_evtQueue.Set();
+ m_evtRemoved.Wait(); // Wait for removal confirmation
}
@@ -102,7 +106,8 @@ void cChunkSender::Execute(void)
while (m_ChunksReady.empty() && m_SendChunks.empty())
{
cCSUnlock Unlock(Lock);
- m_Event.Wait();
+ m_evtRemoved.Set(); // Notify that the removed clients are safe to be deleted
+ m_evtQueue.Wait();
if (mShouldTerminate)
{
return;
@@ -127,6 +132,7 @@ void cChunkSender::Execute(void)
SendChunk(Chunk.m_ChunkX, Chunk.m_ChunkY, Chunk.m_ChunkZ, Chunk.m_Client);
}
+ m_evtRemoved.Set(); // Notify that the removed clients are safe to be deleted
} // while (!mShouldTerminate)
}