Revert "proper finished queue syncing"

This reverts commit 3012b63.

Author: basdxz

src/main/java/com/falsepattern/falsetweaks/modules/threadedupdates/ThreadedChunkUpdateHelper.java

@@ -66,7 +66,6 @@
 import java.util.concurrent.LinkedBlockingDeque;
 import java.util.concurrent.atomic.AtomicBoolean;
 import java.util.concurrent.atomic.AtomicReference;
-import java.util.concurrent.locks.ReentrantLock;
 import java.util.function.Supplier;
 
 import static com.falsepattern.falsetweaks.modules.occlusion.OcclusionRenderer.*;
@@ -105,19 +104,12 @@ public void prepare(List<WorldRenderer> worldRenderersToUpdateList, int updateLi
             worldRenderersToUpdateListInternal.flip();
             back.clear();
             val size = front.size();
-            while (!finishedQueueLock.tryLock()) {
-                Thread.yield();
-            }
-            try {
-                for (int i = 0; i < size; i++) {
-                    val wr = front.get(i);
-                    val task = ((IRendererUpdateResultHolder) wr).ft$getRendererUpdateTask();
-                    if (!task.inFinishedQueue) {
-                        back.add(wr);
-                    }
+            for (int i = 0; i < size; i++) {
+                val wr = front.get(i);
+                val task = ((IRendererUpdateResultHolder) wr).ft$getRendererUpdateTask();
+                if (!task.inFinishedQueue) {
+                    back.add(wr);
                 }
-            } finally {
-                finishedQueueLock.unlock();
             }
             front.clear();
             back.addAll(worldRenderersToUpdateList);
@@ -129,25 +121,18 @@ public void prepare(List<WorldRenderer> worldRenderersToUpdateList, int updateLi
         public boolean hasNext() {
             WorldRenderer wr;
 
-            while (!finishedQueueLock.tryLock()) {
-                Thread.yield();
-            }
-            try {
-                while ((wr = finishedTasks.poll()) != null) {
-                    UpdateTask task = ((IRendererUpdateResultHolder) wr).ft$getRendererUpdateTask();
-                    task.inFinishedQueue = false;
-                    if (task.cancelled || (!wr.needsUpdate && !((WorldRendererOcclusion) wr).ft$needsSort())) {
-                        if (AGGRESSIVE_NEODYMIUM_THREADING) {
-                            NeodymiumCompat.safeDiscardTask(task);
-                        }
-                        task.clear();
-                    } else {
-                        nextRenderer = wr;
-                        return true;
+            while ((wr = finishedTasks.poll()) != null) {
+                UpdateTask task = ((IRendererUpdateResultHolder) wr).ft$getRendererUpdateTask();
+                task.inFinishedQueue = false;
+                if (task.cancelled || (!wr.needsUpdate && !((WorldRendererOcclusion) wr).ft$needsSort())) {
+                    if (AGGRESSIVE_NEODYMIUM_THREADING) {
+                        NeodymiumCompat.safeDiscardTask(task);
                     }
+                    task.clear();
+                } else {
+                    nextRenderer = wr;
+                    return true;
                 }
-            } finally {
-                finishedQueueLock.unlock();
             }
             return false;
         }
@@ -397,24 +382,17 @@ private void updateWorkQueue(List<WorldRenderer> toUpdateList, int updateLimit)
     }
 
     private void removeCancelledResults() {
-        while (!finishedQueueLock.tryLock()) {
-            Thread.yield();
-        }
-        try {
-            for (Iterator<WorldRenderer> it = finishedTasks.iterator(); it.hasNext(); ) {
-                WorldRenderer wr = it.next();
-                UpdateTask task = ((IRendererUpdateResultHolder) wr).ft$getRendererUpdateTask();
-                if (task.cancelled) {
-                    // Discard results and allow re-schedule on worker thread.
-                    if (AGGRESSIVE_NEODYMIUM_THREADING) {
-                        NeodymiumCompat.safeDiscardTask(task);
-                    }
-                    task.clear();
-                    it.remove();
+        for (Iterator<WorldRenderer> it = finishedTasks.iterator(); it.hasNext(); ) {
+            WorldRenderer wr = it.next();
+            UpdateTask task = ((IRendererUpdateResultHolder) wr).ft$getRendererUpdateTask();
+            if (task.cancelled) {
+                // Discard results and allow re-schedule on worker thread.
+                if (AGGRESSIVE_NEODYMIUM_THREADING) {
+                    NeodymiumCompat.safeDiscardTask(task);
                 }
+                task.clear();
+                it.remove();
             }
-        } finally {
-            finishedQueueLock.unlock();
         }
     }
 
@@ -752,22 +730,13 @@ private void runTask(WorldRenderer wr, Profiler profiler) {
             task.clear();
         }
         ((ICapturableTessellator) threadTessellator.get()).discard();
-        while (!finishedQueueLock.tryLock()) {
-            Thread.yield();
-        }
-        try {
-            finishedTasks.add(wr);
-            task.inFinishedQueue = true;
-        } finally {
-            finishedQueueLock.unlock();
-        }
+        finishedTasks.add(wr);
+        task.inFinishedQueue = true;
         } finally {
             profiler.endSection();
         }
     }
 
-    private final ReentrantLock finishedQueueLock = new ReentrantLock();
-
     private class WorkerThread extends FastThreadLocal.TurboThread {
         private final AtomicBoolean myRun;
         private final CircularTaskQueue taskQueue;