godotengine
/
godot
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Main::cleanup: Move MessageQueue deletion further down where it's safer 

Partial revert of #15702 which triggered the issue.

Fixes #39786.
This commit is contained in:
Rémi Verschelde 2020-06-24 16:57:37 +02:00
parent 4c0b077faa
commit 6480efba45
1 changed files with 5 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
main/main.cpp
View File

@ -2321,8 +2321,8 @@ void Main::cleanup() {
ResourceLoader::remove_custom_loaders(); ResourceLoader::remove_custom_loaders();
ResourceSaver::remove_custom_savers(); ResourceSaver::remove_custom_savers();
// Flush before uninitializing the scene, but delete the MessageQueue as late as possible.
message_queue->flush(); message_queue->flush();
memdelete(message_queue);
OS::get_singleton()->delete_main_loop(); OS::get_singleton()->delete_main_loop();
@ -2408,6 +2408,10 @@ void Main::cleanup() {
OS::get_singleton()->set_restart_on_exit(false, List<String>()); //clear list (uses memory) OS::get_singleton()->set_restart_on_exit(false, List<String>()); //clear list (uses memory)
} }
// Now should be safe to delete MessageQueue (famous last words).
message_queue->flush();
memdelete(message_queue);
unregister_core_driver_types(); unregister_core_driver_types();
unregister_core_types(); unregister_core_types();