Revert "Add buffering for non-immediate priority packets to reduce compression overhead"

This reverts commit 2162675b6400807cf650f6fc7ad236c52fa99c01.

This broke Human skins... it seems MCPE doesn't like you adding and
removing a player from the list in the same batch.
This commit is contained in:
Dylan K. Taylor 2017-09-28 14:29:17 +01:00
parent d0bf0ff083
commit 9bc8d8db79

View File

@ -1064,10 +1064,6 @@ class Player extends Human implements CommandSender, ChunkLoader, IPlayer{
return false;
}
if(!$packet->canBeBatched()){
throw new \InvalidArgumentException(get_class($packet) . " cannot be added to batch buffer");
}
$timings = Timings::getSendDataPacketTimings($packet);
$timings->startTiming();
$this->server->getPluginManager()->callEvent($ev = new DataPacketSendEvent($this, $packet));
@ -1126,12 +1122,6 @@ class Player extends Human implements CommandSender, ChunkLoader, IPlayer{
return false;
}
if(!$needACK and !$immediate and $packet->canBeBatched()){
$this->batchedPackets[] = clone $packet;
return true;
}
$identifier = $this->interface->putPacket($this, $packet, $needACK, $immediate);
if($needACK and $identifier !== null){