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

fix video crash when exiting state #16300

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Conversation

inky03
Copy link
Contributor

@inky03 inky03 commented Feb 13, 2025

simply separates the part that calls the VideoSprite finish callback so it isnt called when the video is just destroyed, which for instance causes a crash in PlayState

@Rom4chek
Copy link

this fix crashed for me once it tried to do function "remove(cover)"

@inky03
Copy link
Contributor Author

inky03 commented Feb 13, 2025

this fix crashed for me once it tried to do function "remove(cover)"

just edited a part in the code, but i cant find a way to reproduce it yet. maybe more details would be useful

@Rom4chek
Copy link

Ok edit you made doesnt make it crash now, yay

Copy link

@KELOTOMU KELOTOMU left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sure why not

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