makes more sense

Author: mrsterner

Diff for: src/main/java/dev/sterner/mixin/ReboundEnchantmentMixin.java

@@ -19,8 +19,7 @@ public class ReboundEnchantmentMixin {
     private static void voidbound$onRightClick(ServerPlayer player, InteractionHand interactionHand, ItemStack stack, CallbackInfoReturnable<Boolean> cir, @Local(ordinal = 0) float baseDamage, @Local(ordinal = 1) float magicDamage) {
 
         if (stack.getItem() instanceof IchoriumScytheItem) {
-            ReboundEnchantmentMixinLogic.INSTANCE.logic(player, stack, baseDamage, magicDamage, 25);
-            ReboundEnchantmentMixinLogic.INSTANCE.logic(player, stack, baseDamage, magicDamage, -25);
+            ReboundEnchantmentMixinLogic.INSTANCE.logic(player, stack, baseDamage, magicDamage);
         }
     }
 }

Diff for: src/main/kotlin/dev/sterner/mixin_logic/ReboundEnchantmentMixinLogic.kt

@@ -8,7 +8,12 @@ import net.minecraft.world.item.ItemStack
 
 object ReboundEnchantmentMixinLogic {
 
-    fun logic(player: ServerPlayer, stack: ItemStack, baseDamage: Float, magicDamage: Float, yOffset: Int) {
+    fun logic(player: ServerPlayer, stack: ItemStack, baseDamage: Float, magicDamage: Float) {
+        shootScythe(player, stack, baseDamage, magicDamage, 25)
+        shootScythe(player, stack, baseDamage, magicDamage, -25)
+    }
+
+    private fun shootScythe(player: ServerPlayer, stack: ItemStack, baseDamage: Float, magicDamage: Float, yRotOffset: Int) {
         val entity = ScytheBoomerangEntity(
             player.level(),
             player.position().x,
@@ -22,7 +27,7 @@ object ReboundEnchantmentMixinLogic {
         (entity as IchoriumScytheGhost).setGhost(true)
 
         entity.shootFromRotation(
-            player, player.xRot, player.yRot + yOffset, 0.0f,
+            player, player.xRot, player.yRot + yRotOffset, 0.0f,
             (1.5f + player.getAttributeValue(AttributeRegistry.SCYTHE_PROFICIENCY.get()) * 0.125f).toFloat(), 0f
         )
         player.level().addFreshEntity(entity)