diff options
| author | Rémi Verschelde <rverschelde@gmail.com> | 2020-05-03 09:58:07 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2020-05-03 09:58:07 +0200 |
| commit | 3c200b23fbf43adae567b0a61052b396ae03b16d (patch) | |
| tree | 3d12fba5ccb18edadc9019e268e935f8df7d38aa /modules/gdscript/gdscript.cpp | |
| parent | 1309bfa97a0f2c2a8db004d7107b1c9a769d40fb (diff) | |
| parent | 7001d06f9d7e11e042147899acbdd01c13ad8983 (diff) | |
| download | redot-engine-3c200b23fbf43adae567b0a61052b396ae03b16d.tar.gz | |
Merge pull request #32534 from Xrayez/fix-dict2inst-init
Make `dict2inst` to work with arbitrary `_init` parameters
Diffstat (limited to 'modules/gdscript/gdscript.cpp')
| -rw-r--r-- | modules/gdscript/gdscript.cpp | 11 |
1 files changed, 4 insertions, 7 deletions
diff --git a/modules/gdscript/gdscript.cpp b/modules/gdscript/gdscript.cpp index 0316581604..06ab9e226d 100644 --- a/modules/gdscript/gdscript.cpp +++ b/modules/gdscript/gdscript.cpp @@ -103,15 +103,14 @@ GDScriptInstance *GDScript::_create_instance(const Variant **p_args, int p_argco instance->owner->set_script_instance(instance); /* STEP 2, INITIALIZE AND CONSTRUCT */ - { MutexLock lock(GDScriptLanguage::singleton->lock); - instances.insert(instance->owner); } - + if (p_argcount < 0) { + return instance; + } initializer->call(instance, p_args, p_argcount, r_error); - if (r_error.error != Callable::CallError::CALL_OK) { instance->script = Ref<GDScript>(); instance->owner->set_script_instance(nullptr); @@ -119,10 +118,8 @@ GDScriptInstance *GDScript::_create_instance(const Variant **p_args, int p_argco MutexLock lock(GDScriptLanguage::singleton->lock); instances.erase(p_owner); } - - ERR_FAIL_COND_V(r_error.error != Callable::CallError::CALL_OK, nullptr); //error constructing + ERR_FAIL_V_MSG(nullptr, "Error constructing a GDScriptInstance."); } - //@TODO make thread safe return instance; } |