perf: MultiplexTransport: avoid Linq allocations that would happen on…

… every packet send because Send calls .ServerActive() each time
MirageNet · Oct 30, 2019 · 7fe8888 · 7fe8888
1 parent 40f50ac
commit 7fe8888
Showing 1 changed file with 9 additions and 2 deletions.
diff --git a/Assets/Mirror/Runtime/Transport/MultiplexTransport.cs b/Assets/Mirror/Runtime/Transport/MultiplexTransport.cs
@@ -1,6 +1,5 @@
 using System;
 using System.Collections.Generic;
-using System.Linq;
 using System.Text;
 using UnityEngine;
 
@@ -138,7 +137,15 @@ void InitServer()
 
         public override bool ServerActive()
         {
-            return transports.All(t => t.ServerActive());
+            // avoid Linq.All allocations
+            foreach (Transport transport in transports)
+            {
+                if (!transport.ServerActive())
+                {
+                    return false;
+                }
+            }
+            return true;
         }
 
         public override string ServerGetClientAddress(int connectionId)