Fixed Manager memory leak while reconnecting in .NET 4.5 (#108)

blurt · mattqs · commit 6260d78bba92 · 2017-07-15T08:23:22.000+10:00
diff --git a/Src/SocketIoClientDotNet.net45/Client/Manager.cs b/Src/SocketIoClientDotNet.net45/Client/Manager.cs
@@ -401,6 +401,10 @@ private void Cleanup()
             {
                 sub.Destroy();
             }
+
+            // dequeue until empty (ConcurrentQueue does not have a Clear() method)
+            On.IHandle tmp;
+            while (Subs.TryDequeue(out tmp));
         }
 
         public void Close()

Original file line number	Diff line number	Diff line change
`@@ -401,6 +401,10 @@ private void Cleanup()`
`401`	`401`	`{`
`402`	`402`	`sub.Destroy();`
`403`	`403`	`}`
	`404`	`+`
	`405`	`+ // dequeue until empty (ConcurrentQueue does not have a Clear() method)`
	`406`	`+ On.IHandle tmp;`
	`407`	`+ while (Subs.TryDequeue(out tmp));`
`404`	`408`	`}`
`405`	`409`
`406`	`410`	`public void Close()`