Merge pull request #10085 from endragor/no-uninitialized-cb

Dont call nativescript callbacks if lib is not initialized
This commit is contained in:
Thomas Herzog 2017-08-04 16:59:06 +02:00 committed by GitHub
commit f79a5c464b
3 changed files with 17 additions and 10 deletions

View file

@ -155,7 +155,6 @@ String GDNativeLibrary::get_active_library_path() const {
}
GDNative::GDNative() {
initialized = false;
native_handle = NULL;
}
@ -219,6 +218,9 @@ bool GDNative::initialize() {
library_init);
if (err || !library_init) {
OS::get_singleton()->close_dynamic_library(native_handle);
native_handle = NULL;
ERR_PRINT("Failed to obtain godot_gdnative_init symbol");
return false;
}
@ -272,7 +274,11 @@ bool GDNative::terminate() {
OS::get_singleton()->close_dynamic_library(native_handle);
native_handle = NULL;
return false;
return true;
}
bool GDNative::is_initialized() {
return (native_handle != NULL);
}
void GDNativeCallRegistry::register_native_call_type(StringName p_call_type, native_call_cb p_callback) {

View file

@ -117,7 +117,6 @@ class GDNative : public Reference {
GDCLASS(GDNative, Reference)
Ref<GDNativeLibrary> library;
bool initialized;
// TODO(karroffel): different platforms? WASM????
void *native_handle;

View file

@ -1055,13 +1055,15 @@ void NativeScriptLanguage::unregister_script(NativeScript *script) {
void NativeScriptLanguage::call_libraries_cb(const StringName &name) {
// library_gdnatives is modified only from the main thread, so it's safe not to use mutex here
for (Map<String, Ref<GDNative> >::Element *L = library_gdnatives.front(); L; L = L->next()) {
L->get()->call_native_raw(
_noarg_call_type,
name,
NULL,
0,
NULL,
NULL);
if (L->get()->is_initialized()) {
L->get()->call_native_raw(
_noarg_call_type,
name,
NULL,
0,
NULL,
NULL);
}
}
}