Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed Fusion memory leak #3644

Merged

Conversation

AsparagusEduardo
Copy link
Collaborator

Description

Fixes memory leak issue with fusions.

Images

6 fusions in a row, compared to before where it started showing graphical issues at the 3rd fusion. Tested with 10 fusions in a row.
mGBA_yEl3RD73h0

Issue(s) that this PR fixes

Fixes memory leak from #3603

Issue(s) that this PR does NOT fix

Item cursor issues from #3603. Even if you cancel using the item before even selecting a mon, the cursor still changes position, so I don't think it's related to the fusion process.

Discord contact info

AsparagusEduardo

@AsparagusEduardo AsparagusEduardo added this to the 1.7.0 milestone Dec 7, 2023
@Jaizu
Copy link

Jaizu commented Dec 7, 2023

loops should be u32 instead of u8 because why not

@Bassoonian Bassoonian merged commit 7016f25 into rh-hideout:upcoming Dec 8, 2023
1 check passed
@Bassoonian Bassoonian mentioned this pull request Dec 8, 2023
PCG06 pushed a commit to PCG06/pokeemerald that referenced this pull request Dec 8, 2023
* Fixed Fusion memory leak

* using u32

---------

Co-authored-by: Bassoonian <[email protected]>
@AsparagusEduardo AsparagusEduardo deleted the RHH/pr/upcoming/FusionFix branch December 8, 2023 12:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants