From: simon <simon@cda61777-01e9-0310-a592-d414129be87e>
Date: Fri, 19 Jan 2001 09:01:50 +0000 (+0000)
Subject: Keepalives are now in seconds not minutes
X-Git-Url: https://git.distorted.org.uk/u/mdw/putty/commitdiff_plain/edce8e45a5bb31bb141734e616de6b4d93202963

Keepalives are now in seconds not minutes


git-svn-id: svn://svn.tartarus.org/sgt/putty@874 cda61777-01e9-0310-a592-d414129be87e
---

diff --git a/putty.h b/putty.h
index 4f9bfa47..d8079899 100644
--- a/putty.h
+++ b/putty.h
@@ -140,7 +140,7 @@ typedef struct {
     enum { PROT_RAW, PROT_TELNET, PROT_SSH } protocol;
     int close_on_exit;
     int warn_on_close;
-    int ping_interval;
+    int ping_interval;                 /* in seconds */
     /* SSH options */
     char remote_cmd[512];
     int nopty;
diff --git a/settings.c b/settings.c
index 235430c0..ada2b740 100644
--- a/settings.c
+++ b/settings.c
@@ -44,7 +44,8 @@ void save_settings (char *section, int do_host, Config *cfg) {
     }
     write_setting_i (sesskey, "CloseOnExit", !!cfg->close_on_exit);
     write_setting_i (sesskey, "WarnOnClose", !!cfg->warn_on_close);
-    write_setting_i (sesskey, "PingInterval", cfg->ping_interval);
+    write_setting_i (sesskey, "PingInterval", cfg->ping_interval / 60);     /* minutes */
+    write_setting_i (sesskey, "PingIntervalSecs", cfg->ping_interval % 60); /* seconds */
     write_setting_s (sesskey, "TerminalType", cfg->termtype);
     write_setting_s (sesskey, "TerminalSpeed", cfg->termspeed);
     {
@@ -167,7 +168,13 @@ void load_settings (char *section, int do_host, Config *cfg) {
 
     gppi (sesskey, "CloseOnExit", 1, &cfg->close_on_exit);
     gppi (sesskey, "WarnOnClose", 1, &cfg->warn_on_close);
-    gppi (sesskey, "PingInterval", 0, &cfg->ping_interval);
+    {
+      /* This is two values for backward compatibility with 0.50/0.51 */
+      int pingmin, pingsec;
+      gppi (sesskey, "PingInterval", 0, &pingmin);
+      gppi (sesskey, "PingIntervalSecs", 0, &pingsec);
+      cfg->ping_interval = pingmin*60 + pingsec;
+    }
     gpps (sesskey, "TerminalType", "xterm", cfg->termtype,
 	  sizeof(cfg->termtype));
     gpps (sesskey, "TerminalSpeed", "38400,38400", cfg->termspeed,
diff --git a/windlg.c b/windlg.c
index dbccb372..96335fda 100644
--- a/windlg.c
+++ b/windlg.c
@@ -960,7 +960,7 @@ static int GenericMainDlgProc (HWND hwnd, UINT msg,
             }
             beginbox(&cp, "Sending of null packets to keep session active",
                      IDC_BOX_CONNECTION2, IDC_BOXT_CONNECTION2);
-            staticedit(&cp, "Minutes between &keepalives (0 to turn off)",
+            staticedit(&cp, "Seconds between &keepalives (0 to turn off)",
                        IDC_PINGSTATIC, IDC_PINGEDIT, 25);
             endbox(&cp);
 
diff --git a/window.c b/window.c
index 26fcb0de..7ad77fec 100644
--- a/window.c
+++ b/window.c
@@ -1077,7 +1077,7 @@ static LRESULT CALLBACK WndProc (HWND hwnd, UINT message,
         {
            time_t now;
            time(&now);
-           if (now-last_movement > cfg.ping_interval * 60 - 10)
+           if (now-last_movement > cfg.ping_interval)
            {
               back->special(TS_PING);
               last_movement = now;