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 initialization of visual server in all platforms #23434

Merged
merged 1 commit into from
Nov 1, 2018

Conversation

vnen
Copy link
Member

@vnen vnen commented Nov 1, 2018

Avoid leaking an extra instance when using threads. Also fix threaded loading issues on Android and iOS.

Fix #21609.
Likely fix #17086 too.

Avoid leaking an extra instance when using threads. Also fix threaded
loading issues on Android and iOS.
@akien-mga akien-mga merged commit 18d06b9 into godotengine:master Nov 1, 2018
@akien-mga
Copy link
Member

Thanks!

akien-mga added a commit that referenced this pull request Nov 1, 2018
Contrarily to what #23434 assumed, this is not a memory leak,
the VisualServerRaster instance is passed as a parameter to
VisualServerWrapMT's constructor.

Fixes #23437.
@akien-mga
Copy link
Member

If cherry-picked, 8ac1bcf should go with it, otherwise it breaks init.

@vnen vnen deleted the visual-thread-fix branch November 1, 2018 18:27
@akien-mga
Copy link
Member

Now that Godot 3.1 has been released, we only plan to cherry-pick critical bug fixes to the 3.0 branch, unless there is very strong support in favor of a specific non-critical issue. Removing cherrypick label for 3.0.

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