* remove warnings

* remove messagebox when computer plays first
* Mulligan input message now reads who is going first (if you go first you are asked to play or draw yet before mulligan)

src/main/java/forge/control/input/InputBlock.java

@@ -22,7 +22,6 @@ import java.util.HashMap;
 import java.util.List;
 
 import forge.Card;
-import forge.Singletons;
 import forge.game.GameState;
 import forge.game.phase.CombatUtil;
 import forge.game.player.Player;

src/main/java/forge/control/input/InputCleanup.java

@@ -18,10 +18,8 @@
 package forge.control.input;
 
 import forge.Card;
-import forge.Singletons;
 import forge.game.GameState;
 import forge.game.player.Player;
-import forge.game.zone.Zone;
 import forge.game.zone.ZoneType;
 import forge.view.ButtonUtil;
 

src/main/java/forge/control/input/InputConfirmMulligan.java

@@ -44,24 +44,25 @@ public class InputConfirmMulligan extends InputSyncronizedBase {
     boolean keepHand = false;
     final boolean isCommander;
     final List<Card> selected = new ArrayList<Card>();
+    final private Player startingPlayer;
     
-    public InputConfirmMulligan(Player humanPlayer, boolean commander) {
+    public InputConfirmMulligan(Player humanPlayer, Player startsGame, boolean commander) {
         super(humanPlayer);
         isCommander = commander;
+        startingPlayer = startsGame; 
     }
     
     /** {@inheritDoc} */
     @Override
     public final void showMessage() {
         GameState game = player.getGame();
-        Player startingPlayer = game.getPhaseHandler().getPlayerTurn();
 
         StringBuilder sb = new StringBuilder();
         if( startingPlayer == player ) {
-            sb.append("You are going first.\n");
+            sb.append("You are going first!\n\n");
         } else {
-            sb.append(startingPlayer.getName()).append(" is going first. ");
+            sb.append(startingPlayer.getName()).append(" is going first.\n");
-            sb.append("You are going ").append(Lang.getOrdinal(game.getPosition(player, startingPlayer))).append(".\n");
+            sb.append("You are going ").append(Lang.getOrdinal(game.getPosition(player, startingPlayer))).append(".\n\n");
         }
 
         if ( isCommander ) {

src/main/java/forge/game/GameAction.java

@@ -1484,7 +1484,7 @@ public class GameAction {
         do {
             for (int i = 0; i < whoCanMulligan.size(); i++) {
                 Player p = whoCanMulligan.get(i);
-                List<Card> toMulligan = p.canMulligan() ? p.getController().getCardsToMulligan(isCommander) : null; 
+                List<Card> toMulligan = p.canMulligan() ? p.getController().getCardsToMulligan(isCommander, firstPlayer) : null; 
                 if ( toMulligan != null ) {
                     if( !isCommander ) {
                         toMulligan = new ArrayList<Card>(p.getCardsIn(ZoneType.Hand));

src/main/java/forge/game/MatchController.java

@@ -20,8 +20,6 @@ import forge.game.player.LobbyPlayer;
 import forge.game.player.LobbyPlayerHuman;
 import forge.game.player.Player;
 import forge.game.player.PlayerStatistics;
-import forge.game.player.PlayerType;
-import forge.gui.GuiDialog;
 import forge.gui.framework.EDocID;
 import forge.gui.framework.SDisplayUtil;
 import forge.gui.match.CMatchUI;
@@ -233,17 +231,13 @@ public class MatchController {
         // per player observers were set in CMatchUI.SINGLETON_INSTANCE.initMatch
     }
 
-    /**
+
-     * TODO: Write javadoc for this method.
+    public void clearGamesPlayed() {
-     */
+        gamesPlayed.clear();
-    public void replayRound() {
-        gamesPlayed.remove(gamesPlayed.size() - 1);
-        startRound();
     }
     
-    public void replay() {
+    public void clearLastGame() {
-        gamesPlayed.clear();
+        gamesPlayed.remove(gamesPlayed.size() - 1);
-        startRound();
     }
 
     /**
@@ -375,9 +369,6 @@ public class MatchController {
         }
         
         boolean willPlay = goesFirst.getController().getWillPlayOnFirstTurn(message);
-        if ( goesFirst.getLobbyPlayer().getType() == PlayerType.COMPUTER ) {
-            GuiDialog.message(message + "\nComputer Going First");
-        }
         goesFirst = willPlay ? goesFirst : goesFirst.getOpponent();
         game.getPhaseHandler().setPlayerTurn(goesFirst);
         return goesFirst;

src/main/java/forge/game/ai/AiInputBlock.java

@@ -3,7 +3,6 @@ package forge.game.ai;
 import java.util.List;
 
 import forge.Card;
-import forge.Singletons;
 import forge.control.input.InputBase;
 import forge.game.GameState;
 import forge.game.phase.CombatUtil;

src/main/java/forge/game/phase/CombatUtil.java

@@ -35,7 +35,6 @@ import forge.CardPredicates;
 import forge.Command;
 import forge.Constant;
 import forge.GameEntity;
-import forge.Singletons;
 import forge.card.CardType;
 import forge.card.MagicColor;
 import forge.card.ability.ApiType;

src/main/java/forge/game/player/Player.java

@@ -53,7 +53,6 @@ import forge.card.spellability.SpellAbility;
 import forge.card.spellability.Target;
 import forge.card.staticability.StaticAbility;
 import forge.card.trigger.TriggerType;
-import forge.control.FControl;
 import forge.game.GameActionUtil;
 import forge.game.GameState;
 import forge.game.GlobalRuleChange;

src/main/java/forge/game/player/PlayerController.java

@@ -139,5 +139,5 @@ public abstract class PlayerController {
     
     public abstract String chooseSomeType(String kindOfType, String aiLogic, List<String> validTypes, List<String> invalidTypes);
     public abstract boolean confirmReplacementEffect(ReplacementEffect replacementEffect, SpellAbility effectSA, String question);
-    public abstract List<Card> getCardsToMulligan(boolean isCommander);
+    public abstract List<Card> getCardsToMulligan(boolean isCommander, Player firstPlayer);
 }

src/main/java/forge/game/player/PlayerControllerAi.java

@@ -288,7 +288,7 @@ public class PlayerControllerAi extends PlayerController {
     }
 
     @Override
-    public List<Card> getCardsToMulligan(boolean isCommander)  { 
+    public List<Card> getCardsToMulligan(boolean isCommander, Player firstPlayer)  { 
         if( !ComputerUtil.wantMulligan(player) )
             return null;
 

src/main/java/forge/game/player/PlayerControllerHuman.java

@@ -469,8 +469,8 @@ public class PlayerControllerHuman extends PlayerController {
 
 
     @Override
-    public List<Card> getCardsToMulligan(boolean isCommander) {
+    public List<Card> getCardsToMulligan(boolean isCommander, Player firstPlayer) {
-        final InputConfirmMulligan inp = new InputConfirmMulligan(player, isCommander);
+        final InputConfirmMulligan inp = new InputConfirmMulligan(player, firstPlayer, isCommander);
         player.getGame().getInputQueue().setInputAndWait(inp);
         return inp.isKeepHand() ? null : inp.getSelectedCards();
     }

src/main/java/forge/gui/match/ControlWinLose.java

@@ -85,7 +85,8 @@ public class ControlWinLose {
     public void actionOnRestart() {
         SOverlayUtils.hideOverlay();
         saveOptions();
-        match.replay();
+        match.clearGamesPlayed();
+        match.startRound();
     }
 
     /** Action performed when "quit" button is pressed in default win/lose UI. */

src/main/java/forge/gui/match/controllers/CDock.java

@@ -33,7 +33,6 @@ import forge.Card;
 import forge.CardLists;
 import forge.CardPredicates.Presets;
 import forge.Command;
-import forge.Singletons;
 import forge.deck.Deck;
 import forge.game.GameState;
 import forge.game.phase.CombatUtil;