test: adding more profiler wrapping

Cammin · May 20, 2024 · 6a16b88 · 6a16b88
1 parent dbd0065
commit 6a16b88
Show file tree

Hide file tree

Showing 3 changed files with 26 additions and 3 deletions.
diff --git a/Assets/LDtkUnity/Editor/ScriptedImporter/LDtkJsonImporter.cs b/Assets/LDtkUnity/Editor/ScriptedImporter/LDtkJsonImporter.cs
@@ -253,16 +253,25 @@ private LDtkTilesetImporter LoadAndCacheTilesetImporter(TilesetDefinition def)
                 return importer;
             }
 
+            LDtkProfiler.BeginSample("TilesetImporterPath");
             string path = TilesetImporterPath(assetPath, def.Identifier);
+            LDtkProfiler.EndSample();
 
-            if (!File.Exists(path))
+            LDtkProfiler.BeginSample("File.Exists");
+            bool exists = File.Exists(path);
+            LDtkProfiler.EndSample();
+
+            if (!exists)
             {
                 Logger.LogError($"Failed to find the required tileset file at \"{path}\". Ensure that LDtk exported a tileset file through a custom command. If the command wasn't configured yet, check the project inspector for more info.");
                 _importersForDefs.Add(def, null);
                 return null;
             }
 
+            LDtkProfiler.BeginSample("GetAtPath");
             importer = (LDtkTilesetImporter)GetAtPath(path);
+            LDtkProfiler.EndSample();
+
             if (importer == null)
             {
                 Logger.LogError($"Failed to load the tileset importer at \"{path}\", but the file exists. The tileset file may have failed to import?");

diff --git a/Assets/LDtkUnity/Editor/ScriptedImporter/LDtkProjectImporter.cs b/Assets/LDtkUnity/Editor/ScriptedImporter/LDtkProjectImporter.cs
@@ -185,9 +185,15 @@ private void MakeDefObjects(LdtkJson json)
         {
             Dictionary<int, LDtkArtifactAssetsTileset> artifacts = MakeTilesetDict(this, json);
 
+            LDtkProfiler.BeginSample("InitializeFromProject");
             DefinitionObjects.InitializeFromProject(json.Defs, artifacts);
+            LDtkProfiler.EndSample();
+
+            LDtkProfiler.BeginSample("Set _definitions");
             _artifacts._definitions = DefinitionObjects.Defs;
+            LDtkProfiler.EndSample();
 
+            LDtkProfiler.BeginSample("AddAllObjectsToAsset");
             foreach (var obj in DefinitionObjects.Defs)
             {
                 if (obj is ILDtkUid uid)
@@ -197,6 +203,7 @@ private void MakeDefObjects(LdtkJson json)
                 }
                 Logger.LogError($"{obj.name} is not a uid! This should never happen", obj);
             }
+            LDtkProfiler.EndSample();
         }
 
         private static void CheckDefaultEditorBehaviour()
@@ -247,12 +254,19 @@ private void BufferEditorCache()
 
         private void MainBuild(LdtkJson json)
         {
+            LDtkProfiler.BeginSample("SetupPreprocessors");
             var preAction = new LDtkAssetProcessorActionCache();
             LDtkAssetProcessorInvoker.AddPreProcessProject(preAction, json, AssetName);
+            LDtkProfiler.EndSample();
+
+            LDtkProfiler.BeginSample("RunPreprocessors");
             preAction.Process();
+            LDtkProfiler.EndSample();
 
+            LDtkProfiler.BeginSample("ImportProject");
             LDtkBuilderProjectFactory factory = new LDtkBuilderProjectFactory(this);
             factory.Import(json);
+            LDtkProfiler.EndSample();
         }
 
         private void TryGenerateEnums(LdtkJson json)

diff --git a/...kUnity/Runtime/Data/Extensions/Definition/ScriptableObjects/LDtkDefinitionObjectsCache.cs b/...kUnity/Runtime/Data/Extensions/Definition/ScriptableObjects/LDtkDefinitionObjectsCache.cs
@@ -33,7 +33,9 @@ internal LDtkDefinitionObjectsCache(LDtkDebugInstance logger)
 
         public void InitializeFromProject(Definitions defs, Dictionary<int, LDtkArtifactAssetsTileset> tilesets)
         {
+            LDtkProfiler.BeginSample("InitializeTilesets");
             InitializeTilesets(tilesets);
+            LDtkProfiler.EndSample();
 
             LDtkProfiler.BeginSample("GenerateObjects");
             GenerateObjects(defs);
@@ -52,8 +54,6 @@ public void InitializeFromProject(Definitions defs, Dictionary<int, LDtkArtifact
             LDtkProfiler.EndSample();
         }
 
-
-
         public void InitializeFromLevel(List<LDtkDefinitionObject> defs, Dictionary<int, LDtkArtifactAssetsTileset> tilesets)
         {
             Profiler.BeginSample("InitializeTilesets");