From 84f6e2f90f6b332caea8b497053ce5366a144527 Mon Sep 17 00:00:00 2001
From: Chandler Cox <mr.chandlercox@gmail.com>
Date: Fri, 17 Nov 2023 11:25:54 -0600
Subject: [PATCH 1/3] Hide search window and search nodes from UIControls

---
 Source/Editor/Windows/Search/ContentSearchWindow.cs | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/Source/Editor/Windows/Search/ContentSearchWindow.cs b/Source/Editor/Windows/Search/ContentSearchWindow.cs
index e6e407247..305fa16d6 100644
--- a/Source/Editor/Windows/Search/ContentSearchWindow.cs
+++ b/Source/Editor/Windows/Search/ContentSearchWindow.cs
@@ -72,6 +72,7 @@ namespace FlaxEngine.Windows.Search
     /// <summary>
     /// The content searching window. Allows to search inside Visual Scripts, Materials, Particles and other assets.
     /// </summary>
+    [HideInEditor]
     internal class ContentSearchWindow : EditorWindow
     {
         /// <summary>
@@ -115,6 +116,7 @@ namespace FlaxEngine.Windows.Search
             }
         }
 
+        [HideInEditor]
         private sealed class SearchResultTreeNode : TreeNode
         {
             public Action<SearchResultTreeNode> Navigate;

From 06c56d0a4b9d84918c75342c9b165369a197aaa0 Mon Sep 17 00:00:00 2001
From: Wojtek Figat <wojtek@figat.pl>
Date: Sat, 18 Nov 2023 23:46:42 +0100
Subject: [PATCH 2/3] Fix bug when spawning new prefab objects during scene
 load due to regression in async scene init

---
 Source/Engine/Level/SceneObjectsFactory.cpp | 20 ++++++++++++--------
 Source/Engine/Level/SceneObjectsFactory.h   |  3 ++-
 2 files changed, 14 insertions(+), 9 deletions(-)

diff --git a/Source/Engine/Level/SceneObjectsFactory.cpp b/Source/Engine/Level/SceneObjectsFactory.cpp
index 954b3b399..574a179d6 100644
--- a/Source/Engine/Level/SceneObjectsFactory.cpp
+++ b/Source/Engine/Level/SceneObjectsFactory.cpp
@@ -48,6 +48,7 @@ void MissingScript::SetReferenceScript(const ScriptingObjectReference<Script>& v
 
 SceneObjectsFactory::Context::Context(ISerializeModifier* modifier)
     : Modifier(modifier)
+    , MainThreadId(Platform::GetCurrentThreadID())
 {
 }
 
@@ -67,10 +68,12 @@ ISerializeModifier* SceneObjectsFactory::Context::GetModifier()
     ISerializeModifier* modifier = Modifier;
     if (Async)
     {
-        // When using context in async then use one ISerializeModifier per-thread
+        // When using context in async then use one ISerializeModifier for each jpb thread
         ISerializeModifier*& modifierThread = Modifiers.Get();
         if (!modifierThread)
         {
+            if (Platform::GetCurrentThreadID() == MainThreadId)
+                return modifier;
             modifierThread = Cache::ISerializeModifier.GetUnscoped();
             Modifiers.Set(modifierThread);
             Locker.Lock();
@@ -84,15 +87,15 @@ ISerializeModifier* SceneObjectsFactory::Context::GetModifier()
     return modifier;
 }
 
-void SceneObjectsFactory::Context::SetupIdsMapping(const SceneObject* obj, ISerializeModifier* modifier)
+void SceneObjectsFactory::Context::SetupIdsMapping(const SceneObject* obj, ISerializeModifier* modifier) const
 {
     int32 instanceIndex;
     if (ObjectToInstance.TryGet(obj->GetID(), instanceIndex) && instanceIndex != modifier->CurrentInstance)
     {
         // Apply the current prefab instance objects ids table to resolve references inside a prefab properly
         modifier->CurrentInstance = instanceIndex;
-        auto& instance = Instances[instanceIndex];
-        for (auto& e : instance.IdsMapping)
+        const auto& instance = Instances[instanceIndex];
+        for (const auto& e : instance.IdsMapping)
             modifier->IdsMapping[e.Key] = e.Value;
     }
 }
@@ -486,9 +489,6 @@ void SceneObjectsFactory::SynchronizeNewPrefabInstances(Context& context, Prefab
             continue;
         const Guid actorParentId = JsonTools::GetGuid(stream, "ParentID");
 
-        // Map prefab object id to this actor so the new objects gets added to it
-        data.Modifier->IdsMapping[actorPrefabObjectId] = actor->GetID();
-
         // Load prefab
         auto prefab = Content::LoadAsync<Prefab>(prefabId);
         if (prefab == nullptr)
@@ -554,8 +554,12 @@ void SceneObjectsFactory::SynchronizeNewPrefabInstances(Context& context, Prefab
             if (spawned)
                 continue;
 
-            // Create instance (including all children)
+            // Map prefab object id to this actor's prefab instance so the new objects gets added to it
+            context.SetupIdsMapping(actor, data.Modifier);
+            data.Modifier->IdsMapping[actorPrefabObjectId] = actor->GetID();
             Scripting::ObjectsLookupIdMapping.Set(&data.Modifier->IdsMapping);
+
+            // Create instance (including all children)
             SynchronizeNewPrefabInstance(context, data, prefab, actor, prefabObjectId);
         }
     }
diff --git a/Source/Engine/Level/SceneObjectsFactory.h b/Source/Engine/Level/SceneObjectsFactory.h
index 97cf6b9ce..f670feba5 100644
--- a/Source/Engine/Level/SceneObjectsFactory.h
+++ b/Source/Engine/Level/SceneObjectsFactory.h
@@ -28,12 +28,13 @@ public:
         Dictionary<Guid, int32> ObjectToInstance;
         CriticalSection Locker;
         ThreadLocal<ISerializeModifier*> Modifiers;
+        uint64 MainThreadId;
 
         Context(ISerializeModifier* modifier);
         ~Context();
 
         ISerializeModifier* GetModifier();
-        void SetupIdsMapping(const SceneObject* obj, ISerializeModifier* modifier);
+        void SetupIdsMapping(const SceneObject* obj, ISerializeModifier* modifier) const;
     };
 
     /// <summary>

From e76fffc95a2cac8ade9a3e7c30c2aec567fb2ad1 Mon Sep 17 00:00:00 2001
From: Wojtek Figat <wojtek@figat.pl>
Date: Sat, 18 Nov 2023 23:59:30 +0100
Subject: [PATCH 3/3] Improve performance of
 06c56d0a4b9d84918c75342c9b165369a197aaa0

---
 Source/Engine/Level/SceneObjectsFactory.cpp | 11 +++++++----
 Source/Engine/Level/SceneObjectsFactory.h   |  1 -
 2 files changed, 7 insertions(+), 5 deletions(-)

diff --git a/Source/Engine/Level/SceneObjectsFactory.cpp b/Source/Engine/Level/SceneObjectsFactory.cpp
index 574a179d6..a46edf070 100644
--- a/Source/Engine/Level/SceneObjectsFactory.cpp
+++ b/Source/Engine/Level/SceneObjectsFactory.cpp
@@ -48,8 +48,9 @@ void MissingScript::SetReferenceScript(const ScriptingObjectReference<Script>& v
 
 SceneObjectsFactory::Context::Context(ISerializeModifier* modifier)
     : Modifier(modifier)
-    , MainThreadId(Platform::GetCurrentThreadID())
 {
+    // Override the main thread value to not create it again in GetModifier() if called from the same thread
+    Modifiers.Set(modifier);
 }
 
 SceneObjectsFactory::Context::~Context()
@@ -59,7 +60,11 @@ SceneObjectsFactory::Context::~Context()
         Array<ISerializeModifier*, FixedAllocation<PLATFORM_THREADS_LIMIT>> modifiers;
         Modifiers.GetValues(modifiers);
         for (ISerializeModifier* e : modifiers)
+        {
+            if (e == Modifier)
+                continue;
             Cache::ISerializeModifier.Put(e);
+        }
     }
 }
 
@@ -68,12 +73,10 @@ ISerializeModifier* SceneObjectsFactory::Context::GetModifier()
     ISerializeModifier* modifier = Modifier;
     if (Async)
     {
-        // When using context in async then use one ISerializeModifier for each jpb thread
+        // When using context in async then use one ISerializeModifier per-thread
         ISerializeModifier*& modifierThread = Modifiers.Get();
         if (!modifierThread)
         {
-            if (Platform::GetCurrentThreadID() == MainThreadId)
-                return modifier;
             modifierThread = Cache::ISerializeModifier.GetUnscoped();
             Modifiers.Set(modifierThread);
             Locker.Lock();
diff --git a/Source/Engine/Level/SceneObjectsFactory.h b/Source/Engine/Level/SceneObjectsFactory.h
index f670feba5..b1a9b8a9d 100644
--- a/Source/Engine/Level/SceneObjectsFactory.h
+++ b/Source/Engine/Level/SceneObjectsFactory.h
@@ -28,7 +28,6 @@ public:
         Dictionary<Guid, int32> ObjectToInstance;
         CriticalSection Locker;
         ThreadLocal<ISerializeModifier*> Modifiers;
-        uint64 MainThreadId;
 
         Context(ISerializeModifier* modifier);
         ~Context();