From c90cda30ce54b1162e2e71fcf5f4f96cc49c5510 Mon Sep 17 00:00:00 2001
From: Sloth <sloth@cardforge.org>
Date: Fri, 16 Nov 2012 17:01:49 +0000
Subject: [PATCH] - Tweaked doTriggerAINoCost of AF DamageAll.

---
 .../card/abilityfactory/ai/DamageAllAi.java   | 55 ++++++++-----------
 1 file changed, 23 insertions(+), 32 deletions(-)

diff --git a/src/main/java/forge/card/abilityfactory/ai/DamageAllAi.java b/src/main/java/forge/card/abilityfactory/ai/DamageAllAi.java
index 48793aa3b7e..2ea7bef1055 100644
--- a/src/main/java/forge/card/abilityfactory/ai/DamageAllAi.java
+++ b/src/main/java/forge/card/abilityfactory/ai/DamageAllAi.java
@@ -167,39 +167,30 @@ public class  DamageAllAi extends SpellAiLogic {
 
         Player enemy = ai.getOpponent();
         final Target tgt = sa.getTarget();
-        do { // A little trick to still check the SubAbilities, once we know we
-             // want to play it
-            if (tgt == null) {
-                // If it's not mandatory check a few things
-                if (mandatory) {
-                    return true;
-                } else {
-                    // Don't get yourself killed
-                    if (validP.contains("Each")
-                            && (ai.getLife() <= ai.predictDamage(dmg,
-                                    source, false))) {
-                        return false;
-                    }
-
-                    // if we can kill human, do it
-                    if ((validP.contains("Each") || validP.contains("EachOpponent") || validP.contains("Targeted"))
-                            && (enemy.getLife() <= enemy.predictDamage(dmg,
-                                    source, false))) {
-                        break;
-                    }
-
-                    // Evaluate creatures getting killed
-                    final List<Card> humanList = this.getKillableCreatures(sa, enemy, dmg);
-                    final List<Card> computerList = this.getKillableCreatures(sa, ai, dmg);
-                    if ((CardFactoryUtil.evaluateCreatureList(computerList) + 50) >= CardFactoryUtil
-                            .evaluateCreatureList(humanList)) {
-                        return false;
-                    }
-                }
-            } else {
-                // DamageAll doesn't really target right now
+        if (tgt == null) {
+            // If it's not mandatory check a few things
+            if (mandatory) {
+                return true;
             }
-        } while (false);
+            // Don't get yourself killed
+            if (validP.contains("Each") && (ai.getLife() <= ai.predictDamage(dmg, source, false))) {
+                return false;
+            }
+
+            // if we can kill human, do it
+            if ((validP.contains("Each") || validP.contains("EachOpponent") || validP.contains("Targeted"))
+                    && (enemy.getLife() <= enemy.predictDamage(dmg, source, false))) {
+                return true;
+            }
+
+            // Evaluate creatures getting killed
+            final List<Card> humanList = this.getKillableCreatures(sa, enemy, dmg);
+            final List<Card> computerList = this.getKillableCreatures(sa, ai, dmg);
+            if (!computerList.isEmpty() && CardFactoryUtil.evaluateCreatureList(computerList) + 50 >= CardFactoryUtil
+                    .evaluateCreatureList(humanList)) {
+                return false;
+            }
+        }
 
         return true;
     }