Added scene cleanup & checks for already running instances

Author: TwoTenPvP

MLAPI/MonoBehaviours/Core/NetworkingManager.cs

@@ -365,6 +365,12 @@ private ConnectionConfig Init()
         /// <param name="netConfig">The NetworkingConfiguration to use</param>
         public void StartServer()
         {
+            if (isServer || isClient)
+            {
+                Debug.LogWarning("MLAPI: Cannot start server while an instance is already running");
+                return;
+            }
+
             ConnectionConfig cConfig = Init();
             if (NetworkConfig.ConnectionApproval)
             {
@@ -389,6 +395,12 @@ public void StartServer()
         /// <param name="netConfig">The NetworkingConfiguration to use</param>
         public void StartClient()
         {
+            if (isServer || isClient)
+            {
+                Debug.LogWarning("MLAPI: Cannot start client while an instance is already running");
+                return;
+            }
+
             ConnectionConfig cConfig = Init();
             HostTopology hostTopology = new HostTopology(cConfig, NetworkConfig.MaxConnections);
             hostId =  NetworkTransport.AddHost(hostTopology, 0, null);
@@ -453,6 +465,11 @@ public void StopClient()
         /// <param name="netConfig">The NetworkingConfiguration to use</param>
         public void StartHost()
         {
+            if (isServer || isClient)
+            {
+                Debug.LogWarning("MLAPI: Cannot start host while an instance is already running");
+                return;
+            }
             ConnectionConfig cConfig = Init();
             if (NetworkConfig.ConnectionApproval)
             {
@@ -502,6 +519,7 @@ private void Shutdown()
             isListening = false;
             _isClient = false;
             _isServer = false;
+            SpawnManager.DestroyNonSceneObjects();
             NetworkTransport.Shutdown();
         }
 

MLAPI/NetworkingManagerComponents/Core/SpawnManager.cs

@@ -75,6 +75,15 @@ internal static void DestroySceneObjects()
             }
         }
 
+        internal static void DestroyNonSceneObjects()
+        {
+            foreach (KeyValuePair<uint, NetworkedObject> netObject in spawnedObjects)
+            {
+                if (!netObject.Value.sceneObject)
+                    MonoBehaviour.Destroy(netObject.Value.gameObject);
+            }
+        }
+
         internal static GameObject SpawnObject(int spawnablePrefabIndex, uint networkId, int ownerId, Vector3 position, Quaternion rotation)
         {
             GameObject go = MonoBehaviour.Instantiate(netManager.SpawnablePrefabs[spawnablePrefabIndex]);