Merge pull request #112 from soopercool101/RelPopulationCheck

Remove unnecessary population check when disposing RELNodes
soopercool101 · Sep 6, 2024 · f0d2be6 · f0d2be6
2 parents 7b5a90c + e01f8c1
commit f0d2be6
Showing 1 changed file with 0 additions and 6 deletions.
diff --git a/BrawlLib/SSBB/ResourceNodes/Modules/RELNode.cs b/BrawlLib/SSBB/ResourceNodes/Modules/RELNode.cs
@@ -297,10 +297,6 @@ public byte? ItemID4
 
         public override void Dispose()
         {
-            while (!_populated)
-            {
-                // Do nothing until population is complete, fixing API enumeration errors
-            }
             _files.Remove(ModuleID);
             base.Dispose();
         }
@@ -372,7 +368,6 @@ public override bool OnInitialize()
 
         private int[] _itemIDOffsets;
 
-        private bool _populated = false;
         public override void OnPopulate()
         {
             _sections = new ModuleSectionNode[_numSections];
@@ -455,7 +450,6 @@ public override void OnPopulate()
 
                 watch.Stop();
                 Console.WriteLine("Took {0} seconds to relocate {1} module", watch.ElapsedMilliseconds / 1000d, Name);
-                _populated = true;
             };
 
             using (BackgroundWorker b = new BackgroundWorker())