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

Merge pull request #46307 from RandomShaper/fix_crash_mono_glue 

Make glue generation shutdown more graceful
This commit is contained in:
Rémi Verschelde 2021-02-22 21:39:01 +01:00 committed by GitHub
parent e1e52b3736 23907e6f19
commit 88ee248db0
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 8 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
main/main.cpp
View File

@ -2576,8 +2576,10 @@ void Main::force_redraw() {
* so that the engine closes cleanly without leaking memory or crashing.
* The order matters as some of those steps are linked with each other.
*/
void Main::cleanup() {
ERR_FAIL_COND(!_start_success);
void Main::cleanup(bool p_force) {
if (!p_force) {
ERR_FAIL_COND(!_start_success);
}
EngineDebugger::deinitialize();

2
main/main.h
View File

@ -59,7 +59,7 @@ public:
static bool is_iterating();
static void cleanup();
static void cleanup(bool p_force = false);
};
// Test main override is for the testing behaviour.

3
modules/mono/editor/bindings_generator.cpp
View File

@ -39,6 +39,7 @@
#include "core/os/file_access.h"
#include "core/os/os.h"
#include "core/string/ucaps.h"
#include "main/main.h"
#include "../glue/cs_glue_version.gen.h"
#include "../godotsharp_defs.h"
@ -3649,6 +3650,7 @@ void BindingsGenerator::handle_cmdline_args(const List<String> &p_cmdline_args)
if (!bindings_generator.initialized) {
ERR_PRINT("Failed to initialize the bindings generator");
Main::cleanup(true);
::exit(0);
}
@ -3675,6 +3677,7 @@ void BindingsGenerator::handle_cmdline_args(const List<String> &p_cmdline_args)
}
// Exit once done
Main::cleanup(true);
::exit(0);
}
}