From 0f2383acc024522bdfc7f96bd1c330178fb3e024 Mon Sep 17 00:00:00 2001 From: Spottedleaf Date: Wed, 6 May 2020 08:07:51 -0700 Subject: [PATCH] Move another NetworkManager util into the inner class (#3303) Those poor souls running outdated ProtocolLib. --- ...k-Manager-and-add-advanced-packet-sup.patch | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/Spigot-Server-Patches/Optimize-Network-Manager-and-add-advanced-packet-sup.patch b/Spigot-Server-Patches/Optimize-Network-Manager-and-add-advanced-packet-sup.patch index a499cb374..1c10af485 100644 --- a/Spigot-Server-Patches/Optimize-Network-Manager-and-add-advanced-packet-sup.patch +++ b/Spigot-Server-Patches/Optimize-Network-Manager-and-add-advanced-packet-sup.patch @@ -70,6 +70,14 @@ index 0000000000000000000000000000000000000000..00000000000000000000000000000000 + } + } + } ++ // Paper start ++ private static boolean canSendImmediate(NetworkManager networkManager, Packet packet) { ++ return networkManager.isPending || networkManager.protocol == EnumProtocol.HANDSHAKING || networkManager.protocol == EnumProtocol.STATUS || networkManager.queueImmunity || ++ packet instanceof PacketPlayOutKeepAlive || ++ packet instanceof PacketPlayOutChat || ++ packet instanceof PacketPlayOutTabComplete; ++ } ++ // Paper end + } + // Paper end @@ -77,14 +85,6 @@ index 0000000000000000000000000000000000000000..00000000000000000000000000000000 this.sendPacket(packet, (GenericFutureListener) null); } -+ // Paper start -+ private boolean canSendImmediate(Packet packet) { -+ return isPending || protocol == EnumProtocol.HANDSHAKING || protocol == EnumProtocol.STATUS || queueImmunity || -+ packet instanceof PacketPlayOutKeepAlive || -+ packet instanceof PacketPlayOutChat || -+ packet instanceof PacketPlayOutTabComplete; -+ } -+ // Paper end public void sendPacket(Packet packet, @Nullable GenericFutureListener> genericfuturelistener) { - if (this.isConnected()) { - this.o(); @@ -94,7 +94,7 @@ index 0000000000000000000000000000000000000000..00000000000000000000000000000000 + // Paper start - handle oversized packets better + boolean connected = this.isConnected(); + if (!connected && !preparing) return; // Do nothing -+ if (connected && (canSendImmediate(packet) || ( ++ if (connected && (InnerUtil.canSendImmediate(this, packet) || ( + MCUtil.isMainThread() && packet.isReady() && this.packetQueue.isEmpty() && + (packet.getExtraPackets() == null || packet.getExtraPackets().isEmpty()) + ))) {