Improve kill effect

src/main/java/ch/njol/skript/effects/EffKill.java

@@ -10,7 +10,6 @@
 import ch.njol.skript.lang.Expression;
 import ch.njol.skript.lang.SkriptParser.ParseResult;
 import ch.njol.util.Kleenean;
-import org.bukkit.GameMode;
 import org.bukkit.entity.Damageable;
 import org.bukkit.entity.EnderDragonPart;
 import org.bukkit.entity.Entity;
@@ -34,36 +33,26 @@ public class EffKill extends Effect {
 		Skript.registerEffect(EffKill.class, "kill %entities%");
 	}
 
-	// Absolutely make sure it dies
-	public static final int DAMAGE_AMOUNT = Integer.MAX_VALUE;
-
 	@SuppressWarnings("null")
 	private Expression<Entity> entities;
 
 	@SuppressWarnings({"unchecked", "null"})
 	@Override
-	public boolean init(final Expression<?>[] exprs, final int matchedPattern, final Kleenean isDelayed, final ParseResult parser) {
+	public boolean init(Expression<?>[] exprs, int matchedPattern, Kleenean isDelayed, ParseResult parser) {
 		entities = (Expression<Entity>) exprs[0];
 		return true;
 	}
 
 	@Override
-	protected void execute(final Event e) {
-		for (Entity entity : entities.getArray(e)) {
+	protected void execute(Event event) {
+		for (Entity entity : entities.getArray(event)) {
 
-			if (entity instanceof EnderDragonPart) {
-				entity = ((EnderDragonPart) entity).getParent();
+			if (entity instanceof EnderDragonPart part) {
+				entity = part.getParent();
 			}
 
-			if (entity instanceof Damageable) {
-				final boolean creative = entity instanceof Player && ((Player) entity).getGameMode() == GameMode.CREATIVE;
-				if (creative) // Set player to survival before applying damage
-					((Player) entity).setGameMode(GameMode.SURVIVAL);
-
-				HealthUtils.damage((Damageable) entity, HealthUtils.getMaxHealth((Damageable) entity) * 100); // just to make sure that it really dies >:)
-
-				if (creative) // Set creative player back to creative
-					((Player) entity).setGameMode(GameMode.CREATIVE);
+			if (entity instanceof Damageable damageable) {
+				HealthUtils.setHealth(damageable, 0);
 			}
 
 			// if everything done so far has failed to kill this thing
@@ -75,8 +64,8 @@ protected void execute(final Event e) {
 	}
 
 	@Override
-	public String toString(final @Nullable Event e, final boolean debug) {
-		return "kill" + entities.toString(e, debug);
+	public String toString(@Nullable Event event, boolean debug) {
+		return "kill " + entities.toString(event, debug);
 	}
 
 }