[gnome-chess] Set the clock properly when loading a timed game

[Michael Catanzaro <mcatanzaro src gnome org>]

commit 61975a9248ff23adddca76b805e3c13a21e51187
Author: Michael Catanzaro <mike catanzaro gmail com>
Date:   Sat Jun 8 16:41:45 2013 -0500

    Set the clock properly when loading a timed game
    
    Previously, we stored the starting time in the PGN in the standardized
    TimeControl field. However this is not sufficient for restarting games
    in progress, as the time remaining for each player is not stored. So we
    must extend PGN to store the time remaining for each player, and look at
    that when starting a game. (Custom PGN extensions are kosher; that's
    what we're already doing to store opposing AI and difficulty.)
    
    https://bugzilla.gnome.org/show_bug.cgi?id=701579

 src/chess-pgn.vala   |   10 ++++++++++
 src/gnome-chess.vala |   40 ++++++++++++++++++++++++++++++++--------
 2 files changed, 42 insertions(+), 8 deletions(-)
---
diff --git a/src/chess-pgn.vala b/src/chess-pgn.vala
index ebce1b9..7c81c73 100644
--- a/src/chess-pgn.vala
+++ b/src/chess-pgn.vala
@@ -104,6 +104,16 @@ public class PGNGame
         get { return tags.lookup ("TimeControl"); }
         set { tags.insert ("TimeControl", value); }
     }
+    public string? white_time_left
+    {
+        get { return tags.lookup ("WhiteTimeLeft"); }
+        set { tags.insert ("WhiteTimeLeft", value); }
+    }
+    public string? black_time_left
+    {
+        get { return tags.lookup ("BlackTimeLeft"); }
+        set { tags.insert ("BlackTimeLeft", value); }
+    }
     public bool set_up
     {
         get { string? v = tags.lookup ("SetUp"); return v != null && v == "1" ? true : false; }
diff --git a/src/gnome-chess.vala b/src/gnome-chess.vala
index 4f3e168..b17ae85 100644
--- a/src/gnome-chess.vala
+++ b/src/gnome-chess.vala
@@ -294,6 +294,7 @@ public class Application : Gtk.Application
             else
                 history.update (game_file, "", pgn_game.result);
             debug ("Writing current game to %s", game_file.get_path ());
+            update_pgn_time_remaining ();
             pgn_game.write (game_file);
         }
         catch (Error e)
@@ -398,15 +399,19 @@ public class Application : Gtk.Application
         }
         game = new ChessGame (fen, moves);
 
-        if (pgn_game.time_control != null)
+        /*
+         * We only support simple timeouts
+         * http://www.saremba.de/chessgml/standards/pgn/pgn-complete.htm#c9.6.1
+         */
+        if (pgn_game.time_control != null && int.parse (pgn_game.time_control) != 0)
         {
-            var controls = pgn_game.time_control.split (":");
-            foreach (var control in controls)
+            if (pgn_game.white_time_left != null && pgn_game.black_time_left != null)
             {
-                /* We only support simple timeouts */
-                var duration = int.parse (control);
-                if (duration > 0)
-                    game.clock = new ChessClock (duration/2, duration/2);
+                var white_seconds = int.parse (pgn_game.white_time_left);
+                var black_seconds = int.parse (pgn_game.black_time_left);
+
+                if (white_seconds > 0 && black_seconds > 0)
+                    game.clock = new ChessClock (white_seconds, black_seconds);
             }
         }
 
@@ -1624,11 +1629,26 @@ public class Application : Gtk.Application
         save_cb (Gtk.ResponseType.OK);
     }
 
+    private void update_pgn_time_remaining ()
+    {
+        if (game.clock != null)
+        {
+            /* We currently only support simple timeouts */
+            uint initial_time = int.parse (pgn_game.time_control);
+            uint white_used = game.clock.white_used_in_seconds;
+            uint black_used = game.clock.black_used_in_seconds;
+
+            pgn_game.white_time_left = (initial_time - white_used).to_string ();
+            pgn_game.black_time_left = (initial_time - black_used).to_string ();
+        }
+    }
+
     private void save_cb (int response_id)
     {
         if (response_id == Gtk.ResponseType.OK)
         {
             save_menu.sensitive = false;
+            update_pgn_time_remaining ();
 
             try
             {
@@ -1727,7 +1747,11 @@ public class Application : Gtk.Application
         pgn_game.time = now.format ("%H:%M:%S");
         var duration = settings.get_int ("duration");
         if (duration > 0)
-            pgn_game.time_control = "%d".printf (duration);
+        {
+            pgn_game.time_control = (duration / 2).to_string ();
+            pgn_game.white_time_left = duration.to_string ();
+            pgn_game.black_time_left = duration.to_string ();
+        }
         var engine_name = settings.get_string ("opponent");
         if (engine_name == "")
         {