World: remove useless internal function

this just complicates the logic for no reason. It had a purpose once, but no longer.
This commit is contained in:
Dylan K. Taylor 2019-05-07 18:51:26 +01:00
parent c1a483a36d
commit 51548d1a27

View File

@ -2416,16 +2416,6 @@ class World implements ChunkManager, Metadatable{
$this->chunkSendQueue[$index][spl_object_id($player)] = $player;
}
private function onChunkReady(int $x, int $z){
if(isset($this->chunkSendQueue[$index = World::chunkHash($x, $z)])){
foreach($this->chunkSendQueue[$index] as $player){
/** @var Player $player */
$player->onChunkReady($x, $z);
}
unset($this->chunkSendQueue[$index]);
}
}
private function processChunkRequests(){
if(count($this->chunkSendQueue) > 0){
$this->timings->syncChunkSendTimer->startTiming();
@ -2439,10 +2429,16 @@ class World implements ChunkManager, Metadatable{
if($chunk === null or !$chunk->isGenerated() or !$chunk->isPopulated()){
throw new ChunkException("Invalid Chunk sent");
}
$this->onChunkReady($x, $z);
foreach($players as $player){
$player->onChunkReady($x, $z);
}
$this->timings->syncChunkSendPrepareTimer->stopTiming();
}
$this->chunkSendQueue = [];
$this->timings->syncChunkSendTimer->stopTiming();
}
}