Allow negative amplifiers in potion effects (#3922)

* Allow negative amplifiers in potion effects * Remove IllegalArgumentException from amplifier signature * run spotlessApply
SpongePowered · Nov 18, 2023 · 335891f · 335891f
1 parent 0b344ee
commit 335891f
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 6 deletions.
diff --git a/src/main/java/org/spongepowered/common/effect/potion/SpongePotionBuilder.java b/src/main/java/org/spongepowered/common/effect/potion/SpongePotionBuilder.java
@@ -117,10 +117,7 @@ public PotionEffect.Builder duration(final @NonNull Ticks duration) {
     }
 
     @Override
-    public PotionEffect.Builder amplifier(final int amplifier) throws IllegalArgumentException {
-        if (amplifier < 0) {
-            throw new IllegalArgumentException("Amplifier must not be negative");
-        }
+    public PotionEffect.Builder amplifier(final int amplifier) {
         this.amplifier = amplifier;
         return this;
     }

diff --git a/...ava/org/spongepowered/common/mixin/core/client/multiplayer/ClientPacketListenerMixin.java b/...ava/org/spongepowered/common/mixin/core/client/multiplayer/ClientPacketListenerMixin.java
@@ -26,9 +26,7 @@
 
 import net.minecraft.client.multiplayer.ClientPacketListener;
 import net.minecraft.network.Connection;
-import org.spongepowered.asm.mixin.Final;
 import org.spongepowered.asm.mixin.Mixin;
-import org.spongepowered.asm.mixin.Shadow;
 import org.spongepowered.common.bridge.network.ConnectionHolderBridge;
 import org.spongepowered.common.mixin.api.minecraft.client.multiplayer.ClientCommonPacketListenerImpl_API;