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 issue causing Godot Android apps / games to freeze on close #85454

Merged
merged 1 commit into from
Nov 28, 2023

Conversation

m4gr3d
Copy link
Contributor

@m4gr3d m4gr3d commented Nov 28, 2023

The issue occurred because during the 'close' event, the logic was trying to terminate the native engine on the UI thread instead of doing on the render thread.

The issue occurred because during the 'close' event, the logic was trying to terminate the native engine on the UI thread instead of doing on the render thread.
@akien-mga akien-mga merged commit d6dd1de into godotengine:master Nov 28, 2023
15 checks passed
@akien-mga
Copy link
Member

Thanks!

@m4gr3d m4gr3d deleted the fix_android_freeze_on_close branch November 28, 2023 07:52
@ZishAan23
Copy link

In which version is this error going to be fixed ?

@YuriSizov
Copy link
Contributor

@ZishAan23 This is a part of Godot 4.2, released 3 weeks ago.

@ZishAan23
Copy link

@YuriSizov if the error is fixed in Godot 4.2 then why my Godot editor is still causing the same issue. One error is fixed that is whenever I want to exit from my game and return to editor I can easily tap the back button and exit. However when an error occurs the same thing happens the back button doesn't work. Is there a new setting or something that I have to change to fix the issue?

@YuriSizov
Copy link
Contributor

@ZishAan23 This PR is addressing a very broad problem of Android apps and games freezing. There can be many reasons for why this happens. Please file a bug report with your example and a reproduction project so it can be debugged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants