diff options
author | Rémi Verschelde <rverschelde@gmail.com> | 2023-06-19 21:17:47 +0200 |
---|---|---|
committer | Rémi Verschelde <rverschelde@gmail.com> | 2023-06-19 21:17:47 +0200 |
commit | 5f9175f969807410bc077fc9caa0fa53febd4319 (patch) | |
tree | 0a470e97452ca298b4405c2a0ffcc53be0294497 /modules/gdscript/gdscript.cpp | |
parent | 7b170d12cf0f8b3a15572fd203aa5cba840975f3 (diff) | |
parent | cbce374f68b1148020786dec24127ccd89209afa (diff) | |
download | redot-engine-5f9175f969807410bc077fc9caa0fa53febd4319.tar.gz |
Merge pull request #76954 from Rindbee/return-null-on-fail-load-script
Returns null and does not cache when the source code of the script fails to load
Diffstat (limited to 'modules/gdscript/gdscript.cpp')
-rw-r--r-- | modules/gdscript/gdscript.cpp | 12 |
1 files changed, 2 insertions, 10 deletions
diff --git a/modules/gdscript/gdscript.cpp b/modules/gdscript/gdscript.cpp index c3547e3db7..0c7d1c0bdb 100644 --- a/modules/gdscript/gdscript.cpp +++ b/modules/gdscript/gdscript.cpp @@ -2596,20 +2596,12 @@ Ref<GDScript> GDScriptLanguage::get_script_by_fully_qualified_name(const String /*************** RESOURCE ***************/ Ref<Resource> ResourceFormatLoaderGDScript::load(const String &p_path, const String &p_original_path, Error *r_error, bool p_use_sub_threads, float *r_progress, CacheMode p_cache_mode) { - if (r_error) { - *r_error = ERR_FILE_CANT_OPEN; - } - Error err; Ref<GDScript> scr = GDScriptCache::get_full_script(p_path, err, "", p_cache_mode == CACHE_MODE_IGNORE); - if (scr.is_null()) { - // Don't fail loading because of parsing error. - scr.instantiate(); - } - if (r_error) { - *r_error = OK; + // Don't fail loading because of parsing error. + *r_error = scr.is_valid() ? OK : err; } return scr; |