just a brace moved, in cases of all(this) the 2nd statement would overide the first one which checks if its an attacker or blocker.

projects/mtg/src/AIPlayer.cpp

@@ -288,7 +288,6 @@ int AIAction::getEfficiency()
             if (target->toughness == 1 && target->getNextOpponent()->power == 1)
                 efficiency += 15;
             //small bonus added for the poor 1/1s, if we can save them, we will unless something else took precidence.
-				}
         //note is the target is being blocked or blocking a creature with wither or deathtouch, it is not even considered for preventing as it is a waste.
         //if its combat blockers, it is being blocked or blocking, and has less prevents the the amount of damage it will be taking, the effeincy is increased slightly and totalled by the danger rank multiplier for final result.
 				int calculateAfterDamage = 0;
@@ -305,6 +304,7 @@ int AIAction::getEfficiency()
 					}
 				}
 				}
+				}
 				//TODO If the card is the target of a damage spell
         break;
     }