Reduce calls to get time

APDevTeam · Jul 16, 2024 · 9f77bc6 · 9f77bc6
1 parent 81859cb
commit 9f77bc6
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 6 deletions.
diff --git a/src/main/java/net/countercraft/movecraft/repair/RepairManager.java b/src/main/java/net/countercraft/movecraft/repair/RepairManager.java
@@ -22,18 +22,22 @@ public void run() {
 
         Set<Repair> completed = new HashSet<>();
         Set<Repair> executed = new HashSet<>();
-        while (System.currentTimeMillis() - start < Config.RepairMaxTickTime) {
+        long time = System.currentTimeMillis();
+        while (time - start < Config.RepairMaxTickTime) {
             Repair repair = repairs.poll();
             if (repair == null)
                 break; // No repairs, jump out
 
-            if (repair.run()) {
+            if (repair.run(time)) {
                 // Repair placed at least a block, return to back of queue
                 executed.add(repair);
             } // Else leave at top of queue
 
-            if (repair.isDone())
+            if (repair.isDone()) {
                 completed.add(repair);
+            }
+
+            time = System.currentTimeMillis();
         }
         repairs.addAll(executed);
 

diff --git a/src/main/java/net/countercraft/movecraft/repair/types/Repair.java b/src/main/java/net/countercraft/movecraft/repair/types/Repair.java
@@ -52,8 +52,8 @@ public int remaining() {
         return queue.size();
     }
 
-    public boolean run() {
-        double elapsedTicks = (System.currentTimeMillis() - lastExecution) * 20.0 / 1000;
+    public boolean run(long time) {
+        double elapsedTicks = (time - lastExecution) * 20.0 / 1000;
         int placedBlocks = 0;
 
         while (elapsedTicks > Config.RepairTicksPerBlock && placedBlocks <= Config.RepairMaxBlocksPerTick) {
@@ -67,7 +67,7 @@ public boolean run() {
         }
 
         if (placedBlocks > 0) {
-            lastExecution = System.currentTimeMillis();
+            lastExecution = time;
             return true;
         }