Merge pull request #288 from kaorahi/ctrlc-sideeffect

Fix: suggestion appears on a existing stone after Ctrl-C
featurecat · Jun 11, 2018 · 29334dd · 29334dd
2 parents 375a5cd + 8eb4b32
commit 29334dd
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/src/main/java/featurecat/lizzie/rules/BoardHistoryList.java b/src/main/java/featurecat/lizzie/rules/BoardHistoryList.java
@@ -29,6 +29,13 @@ public void setGameInfo(GameInfo gameInfo) {
         this.gameInfo = gameInfo;
     }
 
+    public BoardHistoryList shallowCopy() {
+        BoardHistoryList copy = new BoardHistoryList(null);
+        copy.head = head;
+        copy.gameInfo = gameInfo;
+        return copy;
+    }
+
     /**
      * Clear history.
      */

diff --git a/src/main/java/featurecat/lizzie/rules/SGFParser.java b/src/main/java/featurecat/lizzie/rules/SGFParser.java
@@ -217,7 +217,7 @@ public static void save(Board board, String filename) throws IOException {
 
     private static void saveToStream(Board board, Writer writer) throws IOException {
         // collect game info
-        BoardHistoryList history = board.getHistory();
+        BoardHistoryList history = board.getHistory().shallowCopy();
         GameInfo gameInfo = history.getGameInfo();
         String playerBlack = gameInfo.getPlayerBlack();
         String playerWhite = gameInfo.getPlayerWhite();