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

GDExtension: Prevent crash during shutdown as singletons are deleted #91806

Merged
merged 1 commit into from
May 11, 2024

Conversation

dsnopek
Copy link
Contributor

@dsnopek dsnopek commented May 10, 2024

This issue was reported by @mihe on this comment

Marking as draft, because I haven't yet managed to reproduce the issue personally, so I don't know if this fixes it.

@dsnopek dsnopek added bug topic:gdextension crash cherrypick:4.2 Considered for cherry-picking into a future 4.2.x release labels May 10, 2024
@dsnopek dsnopek added this to the 4.3 milestone May 10, 2024
@dsnopek dsnopek requested a review from a team May 10, 2024 17:10
@dsnopek dsnopek requested a review from a team as a code owner May 10, 2024 17:10
@dsnopek dsnopek marked this pull request as draft May 10, 2024 17:10
@dsnopek dsnopek force-pushed the gdextension-shutdown-crash branch from 92e01e2 to 387afda Compare May 10, 2024 17:47
@dsnopek dsnopek force-pushed the gdextension-shutdown-crash branch from 387afda to 14506a4 Compare May 10, 2024 18:17
@dsnopek dsnopek marked this pull request as ready for review May 10, 2024 18:22
@dsnopek
Copy link
Contributor Author

dsnopek commented May 10, 2024

@mihe says this fixed it for him above, so I'm taking it out of draft.

Copy link
Contributor

@mihe mihe left a comment

Choose a reason for hiding this comment

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

LGTM, whatever that's worth. :)

@akien-mga akien-mga merged commit 5e28260 into godotengine:master May 11, 2024
16 checks passed
@akien-mga
Copy link
Member

Thanks!

@dsnopek dsnopek deleted the gdextension-shutdown-crash branch July 22, 2024 15:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug cherrypick:4.2 Considered for cherry-picking into a future 4.2.x release crash topic:gdextension
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants