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

Load 'print_error_with_message' function #1243

Merged
merged 1 commit into from
Sep 15, 2023

Conversation

dsnopek
Copy link
Collaborator

@dsnopek dsnopek commented Sep 15, 2023

I made a dumb mistake in PR #1208 :-(

We do a thing when godot-cpp is initialized, where we load the print_error function first, so we can show errors if any of the other functions fail to load. This means we don't need to load print_error later with the rest of the functions, but instead of removing the load for that, I accidentally removed the load for print_error_with_message, which means it was never getting loaded (and leading to a crash when triggering many of the error macros).

This PR fixes that.

@dsnopek dsnopek added bug This has been identified as a bug crash regression topic:gdextension This relates to the new Godot 4 extension implementation labels Sep 15, 2023
@dsnopek dsnopek added this to the 4.2 milestone Sep 15, 2023
@dsnopek dsnopek requested a review from a team as a code owner September 15, 2023 19:13
@dsnopek dsnopek merged commit 7350096 into godotengine:master Sep 15, 2023
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This has been identified as a bug crash regression topic:gdextension This relates to the new Godot 4 extension implementation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants