Memory and small runtime optimizations

UndertaleModLib/Decompiler/Decompiler.cs

@@ -75,21 +75,32 @@ public DecompileContext(GlobalDecompileContext globalContext, UndertaleCode code
             if (code.ParentEntry != null)
                 throw new InvalidOperationException("This code block represents a function nested inside " + code.ParentEntry.Name + " - decompile that instead");
 
-            if (computeObject && globalContext.Data != null)
+            if (computeObject && globalContext.Data is not null)
             {
                 // TODO: This is expensive, move it somewhere else as a dictionary
                 // and have it update when events/objects are modified.
-                foreach (var obj in globalContext.Data.GameObjects)
-                    foreach (var event_list in obj.Events)
-                        foreach (var subevent in event_list)
-                            foreach (var ev in subevent.Actions)
-                                if (ev.CodeId == code)
-                                {
-                                    Object = obj;
-                                    goto LoopEnd;
-                                }
+
+                // Currently using for loops on purpose, as foreach has memory issues due to IEnumerable
+                for (int i = 0; i < globalContext.Data.GameObjects.Count; i++)
+                {
+                    UndertaleGameObject obj = globalContext.Data.GameObjects[i];
+                    for (int j = 0; j < obj.Events.Count; j++)
+                    {
+                        var eventList = obj.Events[j];
+                        for (int k = 0; k < eventList.Count; k++)
+                        {
+                            UndertaleGameObject.Event subEvent = eventList[k];
+                            for (int l = 0; l < subEvent.Actions.Count; l++)
+                            {
+                                UndertaleGameObject.EventAction ev = subEvent.Actions[l];
+                                if (ev.CodeId != code) continue;
+                                Object = obj;
+                                return;
+                            }
+                        }
+                    }
+                }
             }
-        LoopEnd: return;
         }
 
         #region Struct management
@@ -3464,7 +3475,7 @@ public static Dictionary<Block, List<Block>> ComputeReverseDominators(Dictionary
 
                     Block b = blockList[i];
 
-                    IEnumerable<Block> e;
+                    Block[] e;
                     if (b.conditionalExit)
                     {
                         reverseUse2[0] = b.nextBlockTrue;
@@ -4043,7 +4054,7 @@ public static void BuildSubFunctionCache(UndertaleData data)
             {
                 throw new TimeoutException("The building cache process hung.\n" +
                                            "The function code entries that didn't manage to decompile:\n" +
-                                           String.Join('\n', processingCodeList.Keys) + "\n\n" + 
+                                           String.Join('\n', processingCodeList.Keys) + "\n\n" +
                                            "You should save the game data (if it's necessary) and re-open the app.\n");
             }
         }

UndertaleModLib/Util/DebugUtil.cs

@@ -1,5 +1,6 @@
 using System;
 using System.Collections.Generic;
+using System.Diagnostics;
 using System.Linq;
 using System.Text;
 using System.Threading.Tasks;
@@ -8,6 +9,13 @@ namespace UndertaleModLib.Util
 {
     public class DebugUtil
     {
+
+        /// <summary>
+        /// Asserts that a specified condition is true.
+        /// </summary>
+        /// <param name="expr">The condition to assert for.</param>
+        /// <param name="msg">The message to use if the assertion fails.</param>
+        /// <exception cref="Exception">Gets thrown if the assertion fails.</exception>
         public static void Assert(bool expr, string msg = "Unknown error.")
         {
             if (expr)