checkstyle

src/main/java/forge/card/abilityFactory/AbilityFactory_Attach.java

@@ -233,7 +233,7 @@ public class AbilityFactory_Attach {
         CardList list = AllZoneUtil.getCardsIn(tgt.getZone());
         list = list.getValidCards(tgt.getValidTgts(), sa.getActivatingPlayer(), attachSource);
 
-        // TODO: If Attaching without casting, don't need to actually target.
+        // TODO If Attaching without casting, don't need to actually target.
         // I believe this is the only case where mandatory will be true, so just
         // check that when starting that work
         // But we shouldn't attach to things with Protection
@@ -350,7 +350,7 @@ public class AbilityFactory_Attach {
             return c;
         }
 
-        // TODO: If Not Mandatory, make sure the card is "good enough"
+        // TODO If Not Mandatory, make sure the card is "good enough"
         if (c.isCreature()) {
             int eval = CardFactoryUtil.evaluateCreature(c);
             if (eval < 160 && (eval < 130 || AllZone.getComputerPlayer().getLife() > 5)) {
@@ -511,7 +511,7 @@ public class AbilityFactory_Attach {
         }
 
         if (attachSource.isAura()) {
-            // TODO: For Auras like Rancor, that aren't as likely to lead to
+            // TODO For Auras like Rancor, that aren't as likely to lead to
             // card disadvantage, this check should be skipped
             prefList = prefList.filter(new CardListFilter() {
 

src/main/java/forge/card/abilityFactory/AbilityFactory_ChangeZone.java

@@ -1342,7 +1342,7 @@ public final class AbilityFactory_ChangeZone {
                              {
                                 return false; // don't blink something with
                             }
-                                              // counters TODO: check good and
+                                              // counters TODO check good and
                                               // bad counters
                             return Spell_Permanent.checkETBEffects(c, null, null); // checks
                                                                                    // only

src/main/java/forge/card/abilityFactory/AbilityFactory_CounterMagic.java

@@ -255,7 +255,7 @@ public class AbilityFactory_CounterMagic {
             }
         }
 
-        // TODO: Improve AI
+        // TODO Improve AI
 
         // Will return true if this spell can counter (or is Reusable and can
         // force the Human into making decisions)
@@ -334,7 +334,7 @@ public class AbilityFactory_CounterMagic {
             }
         }
 
-        // TODO: Improve AI
+        // TODO Improve AI
 
         // Will return true if this spell can counter (or is Reusable and can
         // force the Human into making decisions)
@@ -361,7 +361,7 @@ public class AbilityFactory_CounterMagic {
      */
     private void counterResolve(final AbilityFactory af, final SpellAbility sa) {
 
-        // TODO: Before this resolves we should see if any of our targets are
+        // TODO Before this resolves we should see if any of our targets are
         // still on the stack
         ArrayList<SpellAbility> sas;
 

src/main/java/forge/card/abilityFactory/AbilityFactory_Counters.java

@@ -1225,7 +1225,7 @@ public class AbilityFactory_Counters {
             chance &= subAb.chkAI_Drawback();
         }
 
-        // TODO: Make sure Human has poison counters or there are some counters
+        // TODO Make sure Human has poison counters or there are some counters
         // we want to proliferate
         return chance;
     }
@@ -1248,7 +1248,7 @@ public class AbilityFactory_Counters {
             chance &= subAb.doTrigger(mandatory);
         }
 
-        // TODO: Make sure Human has poison counters or there are some counters
+        // TODO Make sure Human has poison counters or there are some counters
         // we want to proliferate
         return chance;
     }

src/main/java/forge/quest/data/QuestData.java

@@ -18,7 +18,6 @@ import forge.properties.NewConstants;
 import forge.quest.data.item.QuestInventory;
 import forge.quest.data.pet.QuestPetManager;
 
-// TODO: Auto-generated Javadoc
 //when you create QuestDataOld and AFTER you copy the AI decks over
 //you have to call one of these two methods below
 //see Gui_QuestOptions for more details

src/main/java/forge/quest/data/item/QuestItemZeppelin.java

@@ -3,7 +3,6 @@ package forge.quest.data.item;
 import forge.AllZone;
 import forge.quest.data.bazaar.QuestStallManager;
 
-// TODO: Auto-generated Javadoc
 /**
  * <p>
  * QuestItemZeppelin class.

src/main/java/forge/quest/gui/bazaar/QuestBazaarStall.java

@@ -13,7 +13,6 @@ import javax.swing.border.EmptyBorder;
 import java.awt.*;
 import java.util.ArrayList;
 
-// TODO: Auto-generated Javadoc
 /**
  * <p>QuestBazaarStall class.</p>
  *