From ed40bc0373fb5e43fd8e3df0a7992c41ca503810 Mon Sep 17 00:00:00 2001
From: CraftBukkit/Spigot <noreply+git-craftbukkit@papermc.io>
Date: Mon, 21 Feb 2011 12:30:45 +1100
Subject: [PATCH] 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.

By: Andrew Ardill <andrew.ardill@gmail.com>
---
 .../org/bukkit/craftbukkit/scheduler/CraftThreadManager.java  | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/paper-server/src/main/java/org/bukkit/craftbukkit/scheduler/CraftThreadManager.java b/paper-server/src/main/java/org/bukkit/craftbukkit/scheduler/CraftThreadManager.java
index d63b458e4..29defcdaa 100644
--- a/paper-server/src/main/java/org/bukkit/craftbukkit/scheduler/CraftThreadManager.java
+++ b/paper-server/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;
     }
 }