summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorTiger Wang <ziwei.tiger@outlook.com>2020-09-03 23:25:35 +0200
committerTiger Wang <ziwei.tiger@outlook.com>2020-09-03 23:25:35 +0200
commit5e670a050bb263ec66227e572eaf9fe0a95ccf1c (patch)
treefbe5e46923e7ccbb33a5857c8bc21787410ca0be
parentDefine _DEBUG macro for tests and main server (#4843) (diff)
downloadcuberite-5e670a050bb263ec66227e572eaf9fe0a95ccf1c.tar
cuberite-5e670a050bb263ec66227e572eaf9fe0a95ccf1c.tar.gz
cuberite-5e670a050bb263ec66227e572eaf9fe0a95ccf1c.tar.bz2
cuberite-5e670a050bb263ec66227e572eaf9fe0a95ccf1c.tar.lz
cuberite-5e670a050bb263ec66227e572eaf9fe0a95ccf1c.tar.xz
cuberite-5e670a050bb263ec66227e572eaf9fe0a95ccf1c.tar.zst
cuberite-5e670a050bb263ec66227e572eaf9fe0a95ccf1c.zip
-rw-r--r--src/ChunkMap.cpp17
-rw-r--r--src/ChunkMap.h5
-rw-r--r--src/World.cpp2
3 files changed, 7 insertions, 17 deletions
diff --git a/src/ChunkMap.cpp b/src/ChunkMap.cpp
index 14d0d0d94..92cce9b55 100644
--- a/src/ChunkMap.cpp
+++ b/src/ChunkMap.cpp
@@ -909,21 +909,12 @@ void cChunkMap::AddEntity(OwnedEntity a_Entity)
-void cChunkMap::AddEntityIfNotPresent(OwnedEntity a_Entity)
+void cChunkMap::AddPlayer(std::unique_ptr<cPlayer> a_Player)
{
cCSLock Lock(m_CSChunks);
- const auto Chunk = FindChunk(a_Entity->GetChunkX(), a_Entity->GetChunkZ());
- if (Chunk == nullptr)
- {
- LOGWARNING("Entity at %p (%s, ID %d) spawning in a non-existent chunk, the entity is lost.",
- static_cast<void *>(a_Entity.get()), a_Entity->GetClass(), a_Entity->GetUniqueID()
- );
- return;
- }
- if (!Chunk->HasEntity(a_Entity->GetUniqueID()))
- {
- Chunk->AddEntity(std::move(a_Entity));
- }
+ auto & Chunk = ConstructChunk(a_Player->GetChunkX(), a_Player->GetChunkZ()); // Always construct the chunk for players
+ ASSERT(!Chunk.HasEntity(a_Player->GetUniqueID()));
+ Chunk.AddEntity(std::move(a_Player));
}
diff --git a/src/ChunkMap.h b/src/ChunkMap.h
index 1b04ae3da..795672135 100644
--- a/src/ChunkMap.h
+++ b/src/ChunkMap.h
@@ -201,9 +201,8 @@ public:
/** Adds the entity to its appropriate chunk, takes ownership of the entity pointer */
void AddEntity(OwnedEntity a_Entity);
- /** Adds the entity to its appropriate chunk, if the entity is not already added.
- Takes ownership of the entity pointer */
- void AddEntityIfNotPresent(OwnedEntity a_Entity);
+ /** Adds the player to its appropriate chunk, takes ownership of the player pointer */
+ void AddPlayer(std::unique_ptr<cPlayer> a_Player);
/** Returns true if the entity with specified ID is present in the chunks */
bool HasEntity(UInt32 a_EntityID) const;
diff --git a/src/World.cpp b/src/World.cpp
index c934c3650..f5b64f297 100644
--- a/src/World.cpp
+++ b/src/World.cpp
@@ -3506,7 +3506,7 @@ void cWorld::AddQueuedPlayers(void)
// Add to chunkmap, if not already there (Spawn vs MoveToWorld):
auto PlayerPtr = Player.get();
- m_ChunkMap->AddEntityIfNotPresent(std::move(Player));
+ m_ChunkMap->AddPlayer(std::move(Player));
PlayerPtr->OnAddToWorld(*this);
ASSERT(!PlayerPtr->IsTicking());
PlayerPtr->SetIsTicking(true);