returning if a thread is not alive should happen before any changes to

the list of workers, hence the return statement should be in the
synchronized block.

1 files changed, 2 insertions, 2 deletions

diff --git a/src/main/java/org/bukkit/craftbukkit/scheduler/CraftThreadManager.java b/src/main/java/org/bukkit/craftbukkit/scheduler/CraftThreadManager.java
index d63b458e..29defcda 100644
--- a/src/main/java/org/bukkit/craftbukkit/scheduler/CraftThreadManager.java
+++ b/src/main/java/org/bukkit/craftbukkit/scheduler/CraftThreadManager.java
@@ -61,8 +61,8 @@ public class CraftThreadManager {
                     return craftWorker.isAlive();
                 }
             }
+            // didn't find it, so it must have been removed
+            return false;
         }
-        // didn't find it, so it must have been removed
-        return false;
     }
 }