Reorder check for isEmpty, on a ConcurrentLinkedQueue its

more expensive than would appear.
This commit is contained in:
md_5 2013-02-12 16:29:15 +11:00
parent e68c3ad981
commit 38fce03c91

View file

@ -1,4 +1,4 @@
From 7bd15ab40724bb5fca76cc4dab281874888dafa1 Mon Sep 17 00:00:00 2001
From 9f9f234ce35b796858b65bb85c5260f2ddd80b16 Mon Sep 17 00:00:00 2001
From: md_5 <md_5@live.com.au>
Date: Sun, 3 Feb 2013 10:24:33 +1100
Subject: [PATCH] Netty
@ -208,7 +208,7 @@ index 0000000..cfc0535
+}
diff --git a/src/main/java/org/spigotmc/netty/NettyNetworkManager.java b/src/main/java/org/spigotmc/netty/NettyNetworkManager.java
new file mode 100644
index 0000000..2d39cc2
index 0000000..81164a4
--- /dev/null
+++ b/src/main/java/org/spigotmc/netty/NettyNetworkManager.java
@@ -0,0 +1,221 @@
@ -379,7 +379,7 @@ index 0000000..2d39cc2
+ }
+
+ // Disconnect via the handler - this performs all plugin related cleanup + logging
+ if (syncPackets.isEmpty() && (dcReason != null || dcArgs != null)) {
+ if ((dcReason != null || dcArgs != null) && syncPackets.isEmpty()) {
+ handler.a(dcReason, dcArgs);
+ }
+ }