Merge pull request #1 from JJ-Atkinson/master

Fix 2 bugs related to AbstractGame.next() and IteratedGame.finished()
nathanmerrill · Feb 24, 2017 · 9a4631f · 9a4631f
2 parents 08929d4 + fb6e26b
commit 9a4631f
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/main/java/com/nmerrill/kothcomm/game/games/AbstractGame.java b/src/main/java/com/nmerrill/kothcomm/game/games/AbstractGame.java
@@ -56,7 +56,7 @@ public final boolean next(){
             return true;
         }
         step();
-        return finished();
+        return !finished();
     }
 
     public abstract boolean finished();

diff --git a/src/main/java/com/nmerrill/kothcomm/game/games/IteratedGame.java b/src/main/java/com/nmerrill/kothcomm/game/games/IteratedGame.java
@@ -23,6 +23,6 @@ protected void step() {
 
     @Override
     public boolean finished() {
-        return iterations >= 0;
+        return iterations <= 0;
     }
 }