Merge pull request #73 from neogeek/hotfix/free-memory

neogeek · web-flow · commit cf2340cf2b75 · 2024-10-14T09:30:00.000-04:00
[hotfix] Added missing call for freeing memory from cpp method.
diff --git a/RhythmGameUtilities/Scripts/Utilities.cs b/RhythmGameUtilities/Scripts/Utilities.cs
@@ -113,6 +113,8 @@ public static List<BeatBar> CalculateBeatBars(Dictionary<int, int> bpmChanges, i
                 beatBars.Add(beatBar);
             }
 
+            Marshal.FreeHGlobal(ptrArray);
+
             return beatBars;
         }
 
diff --git a/UnityPackage/Scripts/Utilities.cs b/UnityPackage/Scripts/Utilities.cs
@@ -113,6 +113,8 @@ public static List<BeatBar> CalculateBeatBars(Dictionary<int, int> bpmChanges, i
                 beatBars.Add(beatBar);
             }
 
+            Marshal.FreeHGlobal(ptrArray);
+
             return beatBars;
         }
 
diff --git a/include/RhythmGameUtilities/Common.hpp b/include/RhythmGameUtilities/Common.hpp
@@ -1,5 +1,6 @@
 #pragma once
 
+#include <cstring>
 #include <regex>
 #include <sstream>
 
diff --git a/include/RhythmGameUtilities/ParsersInternal.hpp b/include/RhythmGameUtilities/ParsersInternal.hpp
@@ -1,5 +1,7 @@
 #pragma once
 
+#include <cstring>
+
 #include "Parsers.hpp"
 
 #ifdef _WIN32

Original file line number	Diff line number	Diff line change
`@@ -113,6 +113,8 @@ public static List<BeatBar> CalculateBeatBars(Dictionary<int, int> bpmChanges, i`
`113`	`113`	`beatBars.Add(beatBar);`
`114`	`114`	`}`
`115`	`115`
	`116`	`+ Marshal.FreeHGlobal(ptrArray);`
	`117`	`+`
`116`	`118`	`return beatBars;`
`117`	`119`	`}`
`118`	`120`