Make requested changes.

Mainly adding Override annotations to methods for interfaces, but a few
other things as well.

Author: kitlith

patchwork-events-lifecycle/src/main/java/net/patchworkmc/mixin/event/lifecycle/MixinMinecraftServer.java

@@ -31,7 +31,7 @@
 @Mixin(MinecraftServer.class)
 public class MixinMinecraftServer {
 	@Inject(method = "run", at = @At(value = "INVOKE", target = "net/minecraft/server/MinecraftServer.exit ()V"))
-	void serverStoppedHook(CallbackInfo ci) {
+	private void serverStoppedHook(CallbackInfo ci) {
 		LifecycleEvents.handleServerStopped((MinecraftServer) (Object) this);
 	}
 }

patchwork-loot/src/main/java/net/patchworkmc/impl/loot/LootHooks.java

@@ -39,6 +39,7 @@
 
 import net.patchworkmc.impl.event.loot.LootEvents;
 
+// NOTE: this class is more or less a direct copy of parts of Forge's ForgeHooks.
 public class LootHooks {
 	@Unique
 	private static ThreadLocal<Deque<LootTableContext>> lootContext = new ThreadLocal<Deque<LootTableContext>>();

patchwork-loot/src/main/java/net/patchworkmc/mixin/loot/MixinLootPool.java

@@ -44,6 +44,7 @@ public class MixinLootPool implements PatchworkLootPool, ForgeLootPool {
 
 	// implementation detail
 	// TODO: if we could have an inner class that was also a mixin, we could set this as protected?
+	@Override
 	public void patchwork$setName(String name) {
 		this.name = name;
 	}
@@ -52,23 +53,28 @@ public class MixinLootPool implements PatchworkLootPool, ForgeLootPool {
 
 	// TODO: freezing stuff
 
+	@Override
 	public String getName() {
 		return this.name;
 	}
 
+	@Override
 	public LootTableRange getRolls() {
 		return rollsRange;
 	}
 
+	@Override
 	public LootTableRange getBonusRolls() {
 		return this.bonusRollsRange;
 	}
 
+	@Override
 	public void setRolls(UniformLootTableRange v) {
 		// checkFrozen();
 		this.rollsRange = v;
 	}
 
+	@Override
 	public void setBonusRolls(UniformLootTableRange v) {
 		// checkFrozen();
 		this.bonusRollsRange = v;

patchwork-loot/src/main/java/net/patchworkmc/mixin/loot/MixinLootPoolBuilder.java

@@ -44,16 +44,15 @@ public abstract class MixinLootPoolBuilder implements ForgeLootPool.Builder {
 	private void addNameToConstructor(CallbackInfoReturnable<LootPool> cir) {
 		LootPool ret = cir.getReturnValue();
 		((PatchworkLootPool) ret).patchwork$setName(name);
-
-		// is this necessary?
-		cir.setReturnValue(ret);
 	}
 
+	@Override
 	public LootPool.Builder name(String name) {
 		this.name = name;
 		return (LootPool.Builder) (Object) this;
 	}
 
+	@Override
 	public LootPool.Builder bonusRolls(float min, float max) {
 		this.bonusRollsRange = new UniformLootTableRange(min, max);
 		return (LootPool.Builder) (Object) this;

patchwork-loot/src/main/java/net/patchworkmc/mixin/loot/MixinLootTable.java

@@ -41,10 +41,12 @@ public class MixinLootTable implements ForgeLootTable {
 
 	// TODO: freezing stuff
 
+	@Override
 	public LootPool getPool(String name) {
 		return ((FabricLootSupplier) this).getPools().stream().filter(e -> name.equals(((ForgeLootPool) e).getName())).findFirst().orElse(null);
 	}
 
+	@Override
 	public LootPool removePool(String name) {
 		// checkFrozen();
 		for (int idx = 0; idx < pools.length; ++idx) {
@@ -60,6 +62,7 @@ public LootPool removePool(String name) {
 		return null;
 	}
 
+	@Override
 	public void addPool(LootPool pool) {
 		// checkFrozen();
 		if (((FabricLootSupplier) this).getPools().stream().anyMatch(e -> e == pool || ((ForgeLootPool) e).getName().equals(((ForgeLootPool) pool).getName()))) {