print_n_panic/Microtransactions64
0
0
Fork 0
mirror of https://github.com/Print-and-Panic/Microtransactions64.git synced 2026-01-21 10:17:19 -08:00
Code Issues Packages Projects Releases Wiki Activity

Fixed memory leak between areas from automatic surface pool size (#452)

Browse Source 
* fixed mem leak

* Add comment to second pop
This commit is contained in:
arthurtilly
2022-07-19 23:24:16 +12:00
committed by GitHub
parent f4b13d474a
commit 1820b146c0
2 changed files with 5 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
src/engine/level_script.c
View File

@@ -360,6 +360,8 @@ static void level_cmd_clear_level(void) {
clear_area_graph_nodes();
clear_areas();
main_pool_pop_state();
// the game does a push on level load and a pop on level unload, we need to add another push to store state after the level has been loaded, so one more pop is needed
main_pool_pop_state();
unmap_tlbs();
sCurrentCmd = CMD_NEXT;
@@ -386,6 +388,7 @@ static void level_cmd_free_level_pool(void) {
break;
}
}
main_pool_push_state();
sCurrentCmd = CMD_NEXT;
}

2
src/game/area.c
View File

@@ -232,6 +232,8 @@ void load_area(s32 index) {
if (gCurrentArea == NULL && gAreaData[index].graphNode != NULL) {
gCurrentArea = &gAreaData[index];
gCurrAreaIndex = gCurrentArea->index;
main_pool_pop_state();
main_pool_push_state();
if (gCurrentArea->terrainData != NULL) {
load_area_terrain(index, gCurrentArea->terrainData, gCurrentArea->surfaceRooms,