Revert "Only check for async callers, do not remove potentially used variables"

This reverts commit da7320fc42.

Spigot-Server-Patches/0111-Check-async-eligible-spawn-chunks-GH-159.patch

@@ -1,21 +0,0 @@
-From 45fbe6bc3de0f99fbbfcca076da14980133b84ef Mon Sep 17 00:00:00 2001
-From: Zach Brown <zach.brown@destroystokyo.com>
-Date: Sun, 27 Mar 2016 20:24:05 -0500
-Subject: [PATCH] Check async, eligible spawn chunks, GH-159
-
-
-diff --git a/src/main/java/net/minecraft/server/SpawnerCreature.java b/src/main/java/net/minecraft/server/SpawnerCreature.java
-index dc7c17c..94b0176 100644
---- a/src/main/java/net/minecraft/server/SpawnerCreature.java
-+++ b/src/main/java/net/minecraft/server/SpawnerCreature.java
-@@ -38,6 +38,7 @@ public final class SpawnerCreature {
-     // Spigot end
- 
-     public int a(WorldServer worldserver, boolean flag, boolean flag1, boolean flag2) {
-+        org.spigotmc.AsyncCatcher.catchOp("check for eligible spawn chunks"); // Paper - At least until we figure out what is calling this async
-         if (!flag && !flag1) {
-             return 0;
-         } else {
--- 
-2.8.0
-

Spigot-Server-Patches/0111-Check-async-remove-unused-vars-GH-159.patch

@@ -0,0 +1,32 @@
+From 7616ed0a0949bef668e0284f5f0f49218d293be7 Mon Sep 17 00:00:00 2001
+From: Zach Brown <zach.brown@destroystokyo.com>
+Date: Sun, 27 Mar 2016 20:24:05 -0500
+Subject: [PATCH] Check async, remove unused vars, GH-159
+
+
+diff --git a/src/main/java/net/minecraft/server/SpawnerCreature.java b/src/main/java/net/minecraft/server/SpawnerCreature.java
+index dc7c17c..15a0ce9 100644
+--- a/src/main/java/net/minecraft/server/SpawnerCreature.java
++++ b/src/main/java/net/minecraft/server/SpawnerCreature.java
+@@ -38,6 +38,7 @@ public final class SpawnerCreature {
+     // Spigot end
+ 
+     public int a(WorldServer worldserver, boolean flag, boolean flag1, boolean flag2) {
++        org.spigotmc.AsyncCatcher.catchOp("check for eligible spawn chunks"); // Paper - At least until we figure out what is calling this async
+         if (!flag && !flag1) {
+             return 0;
+         } else {
+@@ -120,8 +121,10 @@ public final class SpawnerCreature {
+                 // CraftBukkit end
+ 
+                 if ((!enumcreaturetype.d() || flag1) && (enumcreaturetype.d() || flag) && (!enumcreaturetype.e() || flag2)) {
++                    /* Paper start - As far as I can tell neither of these are even used
+                     k = worldserver.a(enumcreaturetype.a());
+                     int l1 = limit * i / a; // CraftBukkit - use per-world limits
++                    */ // Paper end
+ 
+                     if ((mobcnt = getEntityCount(worldserver, enumcreaturetype.a())) <= limit * i / 256) {
+                         BlockPosition.MutableBlockPosition blockposition_mutableblockposition = new BlockPosition.MutableBlockPosition();
+-- 
+2.8.0
+