From b8c207ac2a4ee7d800a41b571fd11b15c4ba6541 Mon Sep 17 00:00:00 2001 From: austinio7116 Date: Thu, 3 May 2018 20:40:59 +0100 Subject: [PATCH] Small improvements to genetic algorithms, merged Agetian's fixes for planar conquest --- .../src/main/java/forge/limited/CardThemedDeckBuilder.java | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/forge-gui/src/main/java/forge/limited/CardThemedDeckBuilder.java b/forge-gui/src/main/java/forge/limited/CardThemedDeckBuilder.java index 9c0f14c622d..272720236e0 100644 --- a/forge-gui/src/main/java/forge/limited/CardThemedDeckBuilder.java +++ b/forge-gui/src/main/java/forge/limited/CardThemedDeckBuilder.java @@ -231,7 +231,9 @@ public class CardThemedDeckBuilder extends DeckGeneratorBase { // 9. If there are still less than 22 non-land cards add off-color // cards. This should be avoided. - addRandomCards(numSpellsNeeded - deckList.size()); + int stillNeeds = numSpellsNeeded - deckList.size(); + if(stillNeeds>0) + addRandomCards(stillNeeds); if (logToConsole) { System.out.println("Post Randoms : " + deckList.size()); }