Merge pull request #35 from Schroedingers-Cat/master

Improve performance of SendMessageToClient() calls (#24)
jorgegarcia · Nov 24, 2018 · d773aaf · d773aaf
2 parents 49218c5 + 7a4f573
commit d773aaf
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/src/OSC/OSCPacket.cs b/src/OSC/OSCPacket.cs
@@ -48,7 +48,7 @@ public string Address
 			}
 			set
 			{
-				Trace.Assert(string.IsNullOrEmpty(_address) == false);
+				Trace.Assert(string.IsNullOrEmpty(value) == false);
 				_address = value;
 			}
 		}

diff --git a/src/OSCHandler.cs b/src/OSCHandler.cs
@@ -157,6 +157,9 @@ void OnApplicationQuit()
 	/// </param>
 	public void CreateClient(string clientId, IPAddress destination, int port)
 	{
+		if (_clients.ContainsKey(clientId)) {
+			return;
+		}
 		ClientLog clientitem = new ClientLog();
 		clientitem.client = new OSCClient(destination, port);
 		clientitem.log = new List<string>();
-Original file line number
+Diff line change
@@ Expand Up / @@ -48,7 +48,7 @@ public string Address @@
     			}
     			set
     			{
-    				Trace.Assert(string.IsNullOrEmpty(_address) == false);
+    				Trace.Assert(string.IsNullOrEmpty(value) == false);
     				_address = value;
     			}
     		}
@@ Expand Down @@