From f1f2336dff1650c55317fda8ecdfb195d2f4b983 Mon Sep 17 00:00:00 2001
From: "Lukas Rieger (Blue)" <TBlueF@users.noreply.github.com>
Date: Mon, 15 May 2023 17:01:10 +0200
Subject: [PATCH] Remove syncronization from estimateProgress() function

---
 .../bluemap/common/rendermanager/CombinedRenderTask.java  | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/BlueMapCommon/src/main/java/de/bluecolored/bluemap/common/rendermanager/CombinedRenderTask.java b/BlueMapCommon/src/main/java/de/bluecolored/bluemap/common/rendermanager/CombinedRenderTask.java
index f7ce290d..e2cb0602 100644
--- a/BlueMapCommon/src/main/java/de/bluecolored/bluemap/common/rendermanager/CombinedRenderTask.java
+++ b/BlueMapCommon/src/main/java/de/bluecolored/bluemap/common/rendermanager/CombinedRenderTask.java
@@ -65,11 +65,13 @@ public class CombinedRenderTask<T extends RenderTask> implements RenderTask {
     }
 
     @Override
-    public synchronized double estimateProgress() {
+    public double estimateProgress() {
         if (!hasMoreWork()) return 1;
+        int currentTask = this.currentTaskIndex;
+        if (currentTask >= this.tasks.size()) return 1;
 
-        double total = currentTaskIndex;
-        total += this.tasks.get(this.currentTaskIndex).estimateProgress();
+        double total = currentTask;
+        total += this.tasks.get(currentTask).estimateProgress();
 
         return total / tasks.size();
     }