The other way leaks memory like a sieve.

src/main/java/com/gmail/nossr50/util/skills/CombatUtils.java

@@ -383,38 +383,40 @@ public final class CombatUtils {
         int numberOfTargets = Misc.getTier(attacker.getItemInHand()); // The higher the weapon tier, the more targets you hit
         int damageAmount = Math.max(damage, 1);
 
-        while (numberOfTargets > 0) {
-            for (Entity entity : target.getNearbyEntities(2.5, 2.5, 2.5)) {
-                if (Misc.isNPCEntity(entity) || !(entity instanceof LivingEntity) || !shouldBeAffected(attacker, entity)) {
-                    continue;
-                }
+        for (Entity entity : target.getNearbyEntities(2.5, 2.5, 2.5)) {
+            if (numberOfTargets <= 0) {
+                break;
+            }
 
-                LivingEntity livingEntity = (LivingEntity) entity;
-                mcMMO.p.getServer().getPluginManager().callEvent(new FakePlayerAnimationEvent(attacker));
+            if (Misc.isNPCEntity(entity) || !(entity instanceof LivingEntity) || !shouldBeAffected(attacker, entity)) {
+                continue;
+            }
 
-                switch (type) {
-                    case SWORDS:
-                        if (entity instanceof Player) {
-                            ((Player) entity).sendMessage(LocaleLoader.getString("Swords.Combat.SS.Struck"));
-                        }
+            LivingEntity livingEntity = (LivingEntity) entity;
+            mcMMO.p.getServer().getPluginManager().callEvent(new FakePlayerAnimationEvent(attacker));
 
-                        BleedTimerTask.add(livingEntity, Swords.serratedStrikesBleedTicks);
-                        break;
+            switch (type) {
+                case SWORDS:
+                    if (entity instanceof Player) {
+                        ((Player) entity).sendMessage(LocaleLoader.getString("Swords.Combat.SS.Struck"));
+                    }
 
-                    case AXES:
-                        if (entity instanceof Player) {
-                            ((Player) entity).sendMessage(LocaleLoader.getString("Axes.Combat.Cleave.Struck"));
-                        }
+                    BleedTimerTask.add(livingEntity, Swords.serratedStrikesBleedTicks);
+                    break;
 
-                        break;
+                case AXES:
+                    if (entity instanceof Player) {
+                        ((Player) entity).sendMessage(LocaleLoader.getString("Axes.Combat.Cleave.Struck"));
+                    }
 
-                    default:
-                        break;
-                }
+                    break;
 
-                dealDamage(livingEntity, damageAmount, attacker);
-                numberOfTargets--;
+                default:
+                    break;
             }
+
+            dealDamage(livingEntity, damageAmount, attacker);
+            numberOfTargets--;
         }
     }