summaryrefslogtreecommitdiffstats
path: root/src/ClientHandle.cpp
diff options
context:
space:
mode:
authorLogicParrot <LogicParrot@users.noreply.github.com>2016-04-05 13:47:16 +0200
committerLogicParrot <LogicParrot@users.noreply.github.com>2016-04-05 13:47:16 +0200
commita6b0c74f79dc6654b67b20c5c77965dccc044412 (patch)
tree555e9d4c2a7c337df931bea567f47d0aef0c6fd8 /src/ClientHandle.cpp
parentMerge pull request #3121 from LogicParrot/loops (diff)
parentImproved player freeze code (diff)
downloadcuberite-a6b0c74f79dc6654b67b20c5c77965dccc044412.tar
cuberite-a6b0c74f79dc6654b67b20c5c77965dccc044412.tar.gz
cuberite-a6b0c74f79dc6654b67b20c5c77965dccc044412.tar.bz2
cuberite-a6b0c74f79dc6654b67b20c5c77965dccc044412.tar.lz
cuberite-a6b0c74f79dc6654b67b20c5c77965dccc044412.tar.xz
cuberite-a6b0c74f79dc6654b67b20c5c77965dccc044412.tar.zst
cuberite-a6b0c74f79dc6654b67b20c5c77965dccc044412.zip
Diffstat (limited to 'src/ClientHandle.cpp')
-rw-r--r--src/ClientHandle.cpp35
1 files changed, 19 insertions, 16 deletions
diff --git a/src/ClientHandle.cpp b/src/ClientHandle.cpp
index 036b0250d..f14ec193e 100644
--- a/src/ClientHandle.cpp
+++ b/src/ClientHandle.cpp
@@ -764,6 +764,12 @@ void cClientHandle::HandlePlayerPos(double a_PosX, double a_PosY, double a_PosZ,
return;
}
+ if (m_Player->IsFrozen())
+ {
+ // Ignore client-side updates if the player is frozen
+ return;
+ }
+
Vector3d NewPosition(a_PosX, a_PosY, a_PosZ);
Vector3d OldPosition = GetPlayer()->GetPosition();
auto PreviousIsOnGround = GetPlayer()->IsOnGround();
@@ -1864,8 +1870,6 @@ void cClientHandle::RemoveFromWorld(void)
// Here, we set last streamed values to bogus ones so everything is resent
m_LastStreamedChunkX = 0x7fffffff;
m_LastStreamedChunkZ = 0x7fffffff;
-
- m_HasSentPlayerChunk = false;
}
@@ -1884,6 +1888,15 @@ void cClientHandle::InvalidateCachedSentChunk()
+bool cClientHandle::IsPlayerChunkSent()
+{
+ return m_HasSentPlayerChunk;
+}
+
+
+
+
+
bool cClientHandle::CheckBlockInteractionsRate(void)
{
ASSERT(m_Player != nullptr);
@@ -1943,8 +1956,9 @@ void cClientHandle::Tick(float a_Dt)
// Freeze the player if it is standing on a chunk not yet sent to the client
+ m_HasSentPlayerChunk = false;
+ if (m_Player->GetParentChunk() != nullptr)
{
- bool PlayerIsStandingAtASentChunk = false;
// If the chunk is invalid, do not bother checking if it's sent to the client, it is definitely not
if (m_Player->GetParentChunk()->IsValid())
{
@@ -1952,7 +1966,7 @@ void cClientHandle::Tick(float a_Dt)
// If so, the chunk has been sent to the client. This is an optimization that saves an iteration of m_SentChunks.
if (cChunkCoords(m_Player->GetChunkX(), m_Player->GetChunkZ()) == m_CachedSentChunk)
{
- PlayerIsStandingAtASentChunk = true;
+ m_HasSentPlayerChunk = true;
}
else
{
@@ -1963,12 +1977,10 @@ void cClientHandle::Tick(float a_Dt)
if (itr != m_SentChunks.end())
{
m_CachedSentChunk = *itr;
- PlayerIsStandingAtASentChunk = true;
+ m_HasSentPlayerChunk = true;
}
}
}
- // The player will freeze itself if it is standing on a chunk not yet sent to the client
- m_Player->TickFreezeCode(PlayerIsStandingAtASentChunk);
}
// If the chunk the player's in was just sent, spawn the player:
@@ -2281,15 +2293,6 @@ void cClientHandle::SendChunkData(int a_ChunkX, int a_ChunkZ, cChunkDataSerializ
cCSLock Lock(m_CSChunkLists);
m_SentChunks.push_back(cChunkCoords(a_ChunkX, a_ChunkZ));
}
-
- // If it is the chunk the player's in, make them spawn (in the tick thread):
- if ((m_State == csAuthenticated) || (m_State == csDownloadingWorld))
- {
- if ((a_ChunkX == m_Player->GetChunkX()) && (a_ChunkZ == m_Player->GetChunkZ()))
- {
- m_HasSentPlayerChunk = true;
- }
- }
}