Merge pull request #57 from chrisprad/Threadleak

mattqs · web-flow · commit a93ce8dcdca5 · 2018-03-25T07:16:02.000+11:00
Fixed runaway threadleak issue due around ping/heartbeats
diff --git a/Src/EngineIoClientDotNet.mono/Client/Socket_net35.cs b/Src/EngineIoClientDotNet.mono/Client/Socket_net35.cs
@@ -566,11 +566,11 @@ private void SetPing()
                 else if (ReadyState == ReadyStateEnum.OPEN)
                 {
                     Ping();
-                    SetHeartbeat(PingTimeout);
                     log2.Info("EasyTimer SetPing finish");
                 }
 
-            }, (int)PingInterval);
+            }, (int)PingInterval);
+            SetHeartbeat(PingTimeout);
         }
 
         private void Ping()

Original file line number	Diff line number	Diff line change
`@@ -566,11 +566,11 @@ private void SetPing()`
`566`	`566`	`else if (ReadyState == ReadyStateEnum.OPEN)`
`567`	`567`	`{`
`568`	`568`	`Ping();`
`569`		`- SetHeartbeat(PingTimeout);`
`570`	`569`	`log2.Info("EasyTimer SetPing finish");`
`571`	`570`	`}`
`572`	`571`
`573`		`- }, (int)PingInterval);`
	`572`	`+ }, (int)PingInterval);`
	`573`	`+ SetHeartbeat(PingTimeout);`
`574`	`574`	`}`
`575`	`575`
`576`	`576`	`private void Ping()`