Fixed issues around errors when starting and stopping playmode in edi…

…tor (#668) * Fixed issues around errors when starting and stopping playmode in editor * updated sdk checkout
XRTK · Sep 28, 2020 · 12e62c6 · 12e62c6
1 parent d512062
commit 12e62c6
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 20 deletions.
diff --git a/Runtime/Providers/SpatialObservers/BaseMixedRealitySpatialMeshObserver.cs b/Runtime/Providers/SpatialObservers/BaseMixedRealitySpatialMeshObserver.cs
@@ -134,10 +134,8 @@ public override void Disable()
             // Disable any spatial meshes we might have.
             foreach (var meshObject in spatialMeshObjects.Values)
             {
-                if (meshObject.GameObject != null)
-                {
-                    meshObject.GameObject.SetActive(false);
-                }
+                Debug.Assert(meshObject.GameObject != null);
+                meshObject.GameObject.SetActive(false);
             }
         }
 
@@ -152,7 +150,7 @@ public override void Destroy()
                 meshObject.GameObject.Destroy();
             }
 
-            spatialMeshObjects.Clear();
+            Debug.Assert(spatialMeshObjects.Count == 0);
 
             lock (spatialMeshObjectPool)
             {
@@ -161,6 +159,8 @@ public override void Destroy()
                     var meshObject = spatialMeshObjectPool.Pop();
                     meshObject.GameObject.Destroy();
                 }
+
+                Debug.Assert(spatialMeshObjectPool.Count == 0);
             }
         }
 

diff --git a/Runtime/Services/MixedRealityToolkit.cs b/Runtime/Services/MixedRealityToolkit.cs
@@ -673,15 +673,7 @@ private void Awake()
 
             if (IsInitialized && instance != this)
             {
-                if (Application.isEditor)
-                {
-                    DestroyImmediate(gameObject);
-                }
-                else
-                {
-                    Destroy(gameObject);
-                }
-
+                gameObject.Destroy();
                 Debug.LogWarning($"Trying to instantiate a second instance of the {nameof(MixedRealityToolkit)}. Additional Instance was destroyed");
             }
             else if (!IsInitialized)

diff --git a/Runtime/Services/SpatialAwarenessSystem/MixedRealitySpatialAwarenessSystem.cs b/Runtime/Services/SpatialAwarenessSystem/MixedRealitySpatialAwarenessSystem.cs
@@ -202,25 +202,22 @@ public override void Destroy()
             if (spatialAwarenessParent != null)
             {
                 spatialAwarenessParent.transform.DetachChildren();
+                spatialAwarenessParent.Destroy();
             }
 
-            spatialAwarenessParent.Destroy();
-
             // Detach the mesh objects (they are to be cleaned up by the observer) and cleanup the parent
             if (meshParent != null)
             {
                 meshParent.transform.DetachChildren();
+                meshParent.Destroy();
             }
 
-            meshParent.Destroy();
-
             // Detach the surface objects (they are to be cleaned up by the observer) and cleanup the parent
             if (surfaceParent != null)
             {
                 surfaceParent.transform.DetachChildren();
+                surfaceParent.Destroy();
             }
-
-            surfaceParent.Destroy();
         }
 
         #region Mesh Events