apply fudge factor to all breaks

Author: beanbag44

common/src/main/kotlin/com/lambda/config/groups/BreakSettings.kt

@@ -31,7 +31,7 @@ class BreakSettings(
     override val breakMode by c.setting("Break Mode", BreakMode.Packet, visibility = vis)
     override val reBreak by c.setting("ReBreak", true, "Re-breaks blocks after they've been broken once", visibility = vis)
     override val unsafeCancels by c.setting("Unsafe Cancels", true, "Allows cancelling block breaking even if the server might continue breaking sever side, potentially causing unexpected state changes", visibility = vis)
-    override val breakThreshold by c.setting("Break Threshold", 0.75f, 0.1f..1.0f, 0.01f, "The break amount at which the block is considered broken", visibility = vis)
+    override val breakThreshold by c.setting("Break Threshold", 0.70f, 0.1f..1.0f, 0.01f, "The break amount at which the block is considered broken", visibility = vis)
     override val doubleBreak by c.setting("Double Break", true, "Allows breaking two blocks at once", visibility = vis)
     override val fudgeFactor by c.setting("Fudge Factor", 2, 0..5, 1, "The amount of ticks to give double, aka secondary breaks extra for the server to recognise the break", visibility = vis)
     override val breakDelay by c.setting("Break Delay", 0, 0..6, 1, "The delay between breaking blocks", " ticks", visibility = vis)

common/src/main/kotlin/com/lambda/interaction/request/breaking/BreakManager.kt

@@ -58,8 +58,8 @@ import net.minecraft.entity.ItemEntity
 import net.minecraft.sound.SoundCategory
 import net.minecraft.util.Hand
 import net.minecraft.util.math.BlockPos
+import kotlin.math.abs
 import kotlin.math.ceil
-import kotlin.math.max
 
 object BreakManager : RequestHandler<BreakRequest>(
     0,
@@ -570,7 +570,7 @@ object BreakManager : RequestHandler<BreakRequest>(
                 val serverBreakTicks = ceil(1.0 / breakDelta).toInt()
                 val clientBreakTicks = ceil(config.breakThreshold / breakDelta).toInt()
                 val diff = serverBreakTicks - clientBreakTicks
-                info.breakingTicks - max(config.fudgeFactor - diff, 0)
+                info.breakingTicks - config.fudgeFactor.coerceAtMost(abs(diff))
             }
         }