Cleanup codebase

Author: Ampflower

src/main/java/tfar/fastbench/MixinHooks.java

@@ -38,7 +38,7 @@
 import tfar.fastbench.interfaces.CraftingInventoryDuck;
 import tfar.fastbench.mixin.ContainerAccessor;
 
-public class MixinHooks {
+public final class MixinHooks {
 
 	public static boolean hascachedrecipe = false;
 

src/main/java/tfar/fastbench/mixin/CraftingContainerMixin.java

@@ -30,19 +30,14 @@
 import net.minecraft.world.Container;
 import net.minecraft.world.inventory.AbstractContainerMenu;
 import net.minecraft.world.inventory.TransientCraftingContainer;
-import org.spongepowered.asm.mixin.Final;
 import org.spongepowered.asm.mixin.Mixin;
-import org.spongepowered.asm.mixin.Shadow;
 import org.spongepowered.asm.mixin.Unique;
 import org.spongepowered.asm.mixin.injection.At;
 import tfar.fastbench.interfaces.CraftingInventoryDuck;
 
 @Mixin(TransientCraftingContainer.class)
 public class CraftingContainerMixin implements CraftingInventoryDuck {
 
-	@Shadow
-	@Final
-	private AbstractContainerMenu menu;
 	@Unique
 	public boolean checkMatrixChanges = true;
 

src/main/java/tfar/fastbench/mixin/CraftingMenuMixin.java

@@ -29,7 +29,6 @@
 import net.minecraft.world.Container;
 import net.minecraft.world.entity.player.Player;
 import net.minecraft.world.inventory.AbstractContainerMenu;
-import net.minecraft.world.inventory.ContainerLevelAccess;
 import net.minecraft.world.inventory.CraftingContainer;
 import net.minecraft.world.inventory.CraftingMenu;
 import net.minecraft.world.inventory.MenuType;
@@ -58,13 +57,6 @@ abstract class CraftingMenuMixin<C extends Container> extends RecipeBookMenu<C>
 	@Shadow
 	@Final
 	private ResultContainer resultSlots;
-	@Shadow
-	@Final
-	private Player player;
-
-	@Shadow
-	@Final
-	private ContainerLevelAccess access;
 
 	protected CraftingMenuMixin(@Nullable MenuType<?> type, int syncId) {
 		super(type, syncId);

src/main/java/tfar/fastbench/mixin/PlayerContainerMixin.java

@@ -29,7 +29,11 @@
 
 import net.minecraft.world.Container;
 import net.minecraft.world.entity.player.Player;
-import net.minecraft.world.inventory.*;
+import net.minecraft.world.inventory.AbstractContainerMenu;
+import net.minecraft.world.inventory.CraftingContainer;
+import net.minecraft.world.inventory.InventoryMenu;
+import net.minecraft.world.inventory.MenuType;
+import net.minecraft.world.inventory.ResultContainer;
 import net.minecraft.world.item.ItemStack;
 import org.jetbrains.annotations.Nullable;
 import org.spongepowered.asm.mixin.Final;
@@ -55,7 +59,7 @@ abstract class PlayerContainerMixin extends AbstractContainerMenu implements Cra
 	@Final
 	private ResultContainer resultSlots;
 
-	@Inject(method = "slotsChanged", at = @At("HEAD"), cancellable = true)
+	@Inject(method = "slotsChanged", at = @At("HEAD"))
 	private void updateResult(Container inventory, CallbackInfo ci) {
 		MixinHooks.slotChangedCraftingGrid(owner.level(), craftSlots, resultSlots);
 	}