From 6e5292c83da8639c826e08fd99f2aa2afdab6d36 Mon Sep 17 00:00:00 2001 From: lupoDharkael Date: Thu, 16 May 2019 13:28:31 +0200 Subject: [PATCH 01/10] Update gitignore --- .gitignore | 156 +++++++++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 156 insertions(+) diff --git a/.gitignore b/.gitignore index 10b327e3..45094ba8 100644 --- a/.gitignore +++ b/.gitignore @@ -16,3 +16,159 @@ bin *.creator.user *.files *.includes + +# Gprof output +gmon.out + +# Vim temp files +*.swo +*.swp + +# Qt project files +*.config +*.creator +*.creator.* +*.files +*.includes +*.cflags +*.cxxflags + +# Eclipse CDT files +.cproject +.settings/ + +# Geany/geany-plugins files +*.geany +.geanyprj + +# Misc +.DS_Store +logs/ + +# for projects that use SCons for building: http://http://www.scons.org/ +.sconf_temp +.sconsign.dblite +*.pyc + +# Visual C++ cache files +ipch/ +*.aps +*.ncb +*.opensdf +*.sdf +*.cachefile +*.VC.db +*.VC.opendb +*.VC.VC.opendb +enc_temp_folder/ + +# Visual Studio profiler +*.psess +*.vsp +*.vspx + +# CodeLite project files +*.project +*.workspace +.codelite/ + +# Windows Azure Build Output +csx/ +*.build.csdef + +# Windows Store app package directory +AppPackages/ + +# Others +sql/ +*.Cache +ClientBin/ +[Ss]tyle[Cc]op.* +~$* +*~ +*.dbmdl +*.dbproj.schemaview +*.pfx +*.publishsettings +node_modules/ +__pycache__/ + +# KDE +.directory + +#Kdevelop project files +*.kdev4 + +# xCode +xcuserdata + +# RIA/Silverlight projects +Generated_Code/ + +# Backup & report files from converting an old project file to a newer +# Visual Studio version. Backup files are not needed, because we have git ;-) +_UpgradeReport_Files/ +Backup*/ +UpgradeLog*.XML +UpgradeLog*.htm + +# SQL Server files +App_Data/*.mdf +App_Data/*.ldf + +# Business Intelligence projects +*.rdl.data +*.bim.layout +*.bim_*.settings + +# Microsoft Fakes +FakesAssemblies/ + +# ========================= +# Windows detritus +# ========================= + +# Windows image file caches +Thumbs.db +ehthumbs.db + +# Folder config file +Desktop.ini + +# Recycle Bin used on file shares +$RECYCLE.BIN/ +logo.h +*.autosave + +# https://github.com/github/gitignore/blob/master/Global/Tags.gitignore +# Ignore tags created by etags, ctags, gtags (GNU global) and cscope +TAGS +!TAGS/ +tags +*.tags +!tags/ +gtags.files +GTAGS +GRTAGS +GPATH +cscope.files +cscope.out +cscope.in.out +cscope.po.out +godot.creator.* + +# Visual Studio 2017 and Visual Studio Code workspace folder +/.vs +/.vscode + +# Visual Studio Code workspace file +*.code-workspace + +# Scons progress indicator +.scons_node_count + +# ccls cache (https://github.com/MaskRay/ccls) +.ccls-cache/ + +# compile commands (https://clang.llvm.org/docs/JSONCompilationDatabase.html) +compile_commands.json From 5c96e5ede5dd9adf0c744c9a425ecb0fc2a2729c Mon Sep 17 00:00:00 2001 From: Duncan Sparks Date: Wed, 17 Jun 2020 11:45:19 -0700 Subject: [PATCH 02/10] Fix alloca.h error in Defs.h --- include/core/Defs.hpp | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/include/core/Defs.hpp b/include/core/Defs.hpp index 1b4923e6..9981afe5 100644 --- a/include/core/Defs.hpp +++ b/include/core/Defs.hpp @@ -62,8 +62,10 @@ enum class Error { #include // alloca() is non-standard. When using MSVC, it's in malloc.h. -#if defined(__linux__) || defined(__APPLE__) || defined(__MINGW32__) +#if defined(__linux__) || defined(__APPLE__) #include +#else +#include #endif typedef float real_t; From 20fdc09c963f876e5d894bc4c0fd3d24b4215190 Mon Sep 17 00:00:00 2001 From: Jummit Date: Wed, 15 Jul 2020 17:05:54 +0200 Subject: [PATCH 03/10] Add missing move_toward to Vector2 and Vector3 --- include/core/Vector2.hpp | 7 +++++++ include/core/Vector3.hpp | 7 +++++++ 2 files changed, 14 insertions(+) diff --git a/include/core/Vector2.hpp b/include/core/Vector2.hpp index 2a4f5fe8..2a4c9027 100644 --- a/include/core/Vector2.hpp +++ b/include/core/Vector2.hpp @@ -178,6 +178,13 @@ struct Vector2 { Vector2 cubic_interpolate(const Vector2 &p_b, const Vector2 &p_pre_a, const Vector2 &p_post_b, real_t p_t) const; + Vector2 move_toward(const Vector2 &p_to, const real_t p_delta) const { + Vector2 v = *this; + Vector2 vd = p_to - v; + real_t len = vd.length(); + return len <= p_delta || len < CMP_EPSILON ? p_to : v + vd / len * p_delta; + } + inline Vector2 slide(const Vector2 &p_vec) const { return p_vec - *this * this->dot(p_vec); } diff --git a/include/core/Vector3.hpp b/include/core/Vector3.hpp index 7e2c3022..bbe8a950 100644 --- a/include/core/Vector3.hpp +++ b/include/core/Vector3.hpp @@ -167,6 +167,13 @@ struct Vector3 { Vector3 cubic_interpolate(const Vector3 &b, const Vector3 &pre_a, const Vector3 &post_b, const real_t t) const; + Vector3 move_toward(const Vector3 &p_to, const real_t p_delta) const { + Vector3 v = *this; + Vector3 vd = p_to - v; + real_t len = vd.length(); + return len <= p_delta || len < CMP_EPSILON ? p_to : v + vd / len * p_delta; + } + Vector3 bounce(const Vector3 &p_normal) const { return -reflect(p_normal); } From 251062c9a5712c43d184c95716fdffeef0417094 Mon Sep 17 00:00:00 2001 From: Marc Gilleron Date: Tue, 28 Jul 2020 18:46:01 +0100 Subject: [PATCH 04/10] Don't use Ref in Object.hpp --- binding_generator.py | 36 ++++++++++++++++++++---------------- 1 file changed, 20 insertions(+), 16 deletions(-) diff --git a/binding_generator.py b/binding_generator.py index 3b01f0ba..13505dbe 100644 --- a/binding_generator.py +++ b/binding_generator.py @@ -46,11 +46,11 @@ def is_reference_type(t): return True return False -def make_gdnative_type(t): +def make_gdnative_type(t, ref_allowed): if is_enum(t): return remove_enum_prefix(t) + " " elif is_class_type(t): - if is_reference_type(t): + if is_reference_type(t) and ref_allowed: return "Ref<" + strip_name(t) + "> " else: return strip_name(t) + " *" @@ -83,8 +83,10 @@ def generate_class_header(used_classes, c): # so don't include it here because it's not needed if class_name != "Object" and class_name != "Reference": source.append("#include ") + ref_allowed = True else: source.append("#include ") + ref_allowed = False included = [] @@ -206,7 +208,7 @@ def generate_class_header(used_classes, c): # TODO decide what to do about virtual methods # method_signature += "virtual " if method["is_virtual"] else "" - method_signature += make_gdnative_type(method["return_type"]) + method_signature += make_gdnative_type(method["return_type"], ref_allowed) method_name = escape_cpp(method["name"]) method_signature += method_name + "(" @@ -215,7 +217,7 @@ def generate_class_header(used_classes, c): method_arguments = "" for i, argument in enumerate(method["arguments"]): - method_signature += "const " + make_gdnative_type(argument["type"]) + method_signature += "const " + make_gdnative_type(argument["type"], ref_allowed) argument_name = escape_cpp(argument["name"]) method_signature += argument_name method_arguments += argument_name @@ -298,6 +300,9 @@ def generate_class_header(used_classes, c): def generate_class_implementation(icalls, used_classes, c): class_name = strip_name(c["name"]) + + ref_allowed = class_name != "Object" and class_name != "Reference" + source = [] source.append("#include \"" + class_name + ".hpp\"") source.append("") @@ -367,12 +372,11 @@ def generate_class_implementation(icalls, used_classes, c): for method in c["methods"]: method_signature = "" - - method_signature += make_gdnative_type(method["return_type"]) + method_signature += make_gdnative_type(method["return_type"], ref_allowed) method_signature += strip_name(c["name"]) + "::" + escape_cpp(method["name"]) + "(" for i, argument in enumerate(method["arguments"]): - method_signature += "const " + make_gdnative_type(argument["type"]) + method_signature += "const " + make_gdnative_type(argument["type"], ref_allowed) method_signature += escape_cpp(argument["name"]) if i != len(method["arguments"]) - 1: @@ -396,12 +400,13 @@ def generate_class_implementation(icalls, used_classes, c): continue return_statement = "" + return_type_is_ref = is_reference_type(method["return_type"]) and ref_allowed if method["return_type"] != "void": if is_class_type(method["return_type"]): if is_enum(method["return_type"]): return_statement += "return (" + remove_enum_prefix(method["return_type"]) + ") " - elif is_reference_type(method["return_type"]): + elif return_type_is_ref: return_statement += "return Ref<" + strip_name(method["return_type"]) + ">::__internal_constructor("; else: return_statement += "return " + ("(" + strip_name(method["return_type"]) + " *) " if is_class_type(method["return_type"]) else "") @@ -476,7 +481,7 @@ def generate_class_implementation(icalls, used_classes, c): if method["return_type"] != "void": cast = "" if is_class_type(method["return_type"]): - if is_reference_type(method["return_type"]): + if return_type_is_ref: cast += "Ref<" + strip_name(method["return_type"]) + ">::__internal_constructor(__result);" else: cast += "(" + strip_name(method["return_type"]) + " *) " + strip_name(method["return_type"] + "::___get_from_variant(") + "__result);" @@ -485,7 +490,6 @@ def generate_class_implementation(icalls, used_classes, c): source.append("\treturn " + cast) - else: args = [] @@ -503,11 +507,15 @@ def generate_class_implementation(icalls, used_classes, c): return_statement += icall_name + "(___mb.mb_" + method["name"] + ", (const Object *) " + core_object_name for arg in method["arguments"]: - return_statement += ", " + escape_cpp(arg["name"]) + (".ptr()" if is_reference_type(arg["type"]) else "") + arg_is_ref = is_reference_type(arg["type"]) and ref_allowed + return_statement += ", " + escape_cpp(arg["name"]) + (".ptr()" if arg_is_ref else "") return_statement += ")" - source.append("\t" + return_statement + (")" if is_reference_type(method["return_type"]) else "") + ";") + if return_type_is_ref: + return_statement += ")" + + source.append("\t" + return_statement + ";") source.append("}") source.append("") @@ -709,7 +717,6 @@ def get_icall_name(sig): - def get_used_classes(c): classes = [] for method in c["methods"]: @@ -725,9 +732,6 @@ def get_used_classes(c): - - - def strip_name(name): if len(name) == 0: return name From a1ba843f3668ed53a25f5eb7866d882552028d53 Mon Sep 17 00:00:00 2001 From: TerraAr <37753757+TerraAr@users.noreply.github.com> Date: Fri, 14 Aug 2020 21:42:10 -0300 Subject: [PATCH 05/10] Created operator. Created String::operator+=(const wchar_t), that was mark as Todo. --- src/core/String.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/core/String.cpp b/src/core/String.cpp index 7340b6d6..65d210fe 100644 --- a/src/core/String.cpp +++ b/src/core/String.cpp @@ -135,7 +135,7 @@ void String::operator+=(const String &s) { } void String::operator+=(const wchar_t c) { - // @Todo + _godot_string = godot::api->godot_string_operator_plus(&_godot_string, &(String(c)._godot_string)); } bool String::operator<(const String &s) const { From 5e656923cfe9469a5f33104503e10ca076ce30b8 Mon Sep 17 00:00:00 2001 From: TerraAr <37753757+TerraAr@users.noreply.github.com> Date: Fri, 14 Aug 2020 23:32:38 -0300 Subject: [PATCH 06/10] Fixed operator Fixed String::operator+=(const wchar_t). The problem was that a temporary variable don't have an address. --- src/core/String.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/core/String.cpp b/src/core/String.cpp index 65d210fe..35da37f8 100644 --- a/src/core/String.cpp +++ b/src/core/String.cpp @@ -135,7 +135,8 @@ void String::operator+=(const String &s) { } void String::operator+=(const wchar_t c) { - _godot_string = godot::api->godot_string_operator_plus(&_godot_string, &(String(c)._godot_string)); + String _to_be_added = c; + _godot_string = godot::api->godot_string_operator_plus(&_godot_string, &_to_be_added._godot_string); } bool String::operator<(const String &s) const { From 0939d0f6d15e8a5870ec7bb0dafb617e0d97d532 Mon Sep 17 00:00:00 2001 From: Daniel Rakos Date: Fri, 21 Jun 2019 16:40:36 +0200 Subject: [PATCH 07/10] Fixed memory leak with String objects The member _godot_string should never be straight out overwritten ever without first destroying the underlying string object's memory. This change solves the problem through the introduction of a new private constructor to create String objects with a pre-existing godot_string handle. --- include/core/String.hpp | 3 + src/core/String.cpp | 174 +++++++++------------------------------- 2 files changed, 40 insertions(+), 137 deletions(-) diff --git a/include/core/String.hpp b/include/core/String.hpp index d448567e..ebf161aa 100644 --- a/include/core/String.hpp +++ b/include/core/String.hpp @@ -29,6 +29,9 @@ public: class String { godot_string _godot_string; + String(godot_string contents) : + _godot_string(contents) {} + public: String(); String(const char *contents); diff --git a/src/core/String.cpp b/src/core/String.cpp index 7340b6d6..0deca29e 100644 --- a/src/core/String.cpp +++ b/src/core/String.cpp @@ -25,52 +25,31 @@ const char *godot::CharString::get_data() const { } String String::num(double p_num, int p_decimals) { - String new_string; - new_string._godot_string = godot::api->godot_string_num_with_decimals(p_num, p_decimals); - - return new_string; + return String(godot::api->godot_string_num_with_decimals(p_num, p_decimals)); } String String::num_scientific(double p_num) { - String new_string; - new_string._godot_string = godot::api->godot_string_num_scientific(p_num); - - return new_string; + return String(godot::api->godot_string_num_scientific(p_num)); } String String::num_real(double p_num) { - String new_string; - new_string._godot_string = godot::api->godot_string_num_real(p_num); - - return new_string; + return String(godot::api->godot_string_num_real(p_num)); } String String::num_int64(int64_t p_num, int base, bool capitalize_hex) { - String new_string; - new_string._godot_string = godot::api->godot_string_num_int64_capitalized(p_num, base, capitalize_hex); - - return new_string; + return String(godot::api->godot_string_num_int64_capitalized(p_num, base, capitalize_hex)); } String String::chr(godot_char_type p_char) { - String new_string; - new_string._godot_string = godot::api->godot_string_chr(p_char); - - return new_string; + return String(godot::api->godot_string_chr(p_char)); } String String::md5(const uint8_t *p_md5) { - String new_string; - new_string._godot_string = godot::api->godot_string_md5(p_md5); - - return new_string; + return String(godot::api->godot_string_md5(p_md5)); } String String::hex_encode_buffer(const uint8_t *p_buffer, int p_len) { - String new_string; - new_string._godot_string = godot::api->godot_string_hex_encode_buffer(p_buffer, p_len); - - return new_string; + return String(godot::api->godot_string_hex_encode_buffer(p_buffer, p_len)); } godot::String::String() { @@ -124,14 +103,11 @@ bool String::operator!=(const String &s) const { } String String::operator+(const String &s) const { - String new_string; - new_string._godot_string = godot::api->godot_string_operator_plus(&_godot_string, &s._godot_string); - - return new_string; + return String(godot::api->godot_string_operator_plus(&_godot_string, &s._godot_string)); } void String::operator+=(const String &s) { - _godot_string = godot::api->godot_string_operator_plus(&_godot_string, &s._godot_string); + *this = String(godot::api->godot_string_operator_plus(&_godot_string, &s._godot_string)); } void String::operator+=(const wchar_t c) { @@ -223,24 +199,15 @@ PoolStringArray String::bigrams() const { } String String::c_escape() const { - String new_string; - new_string._godot_string = godot::api->godot_string_c_escape(&_godot_string); - - return new_string; + return String(godot::api->godot_string_c_escape(&_godot_string)); } String String::c_unescape() const { - String new_string; - new_string._godot_string = godot::api->godot_string_c_unescape(&_godot_string); - - return new_string; + return String(godot::api->godot_string_c_unescape(&_godot_string)); } String String::capitalize() const { - String new_string; - new_string._godot_string = godot::api->godot_string_capitalize(&_godot_string); - - return new_string; + return String(godot::api->godot_string_capitalize(&_godot_string)); } bool String::empty() const { @@ -268,41 +235,31 @@ int String::findn(String p_what, int p_from) const { } String String::format(Variant values) const { - String new_string; - new_string._godot_string = godot::api->godot_string_format(&_godot_string, (godot_variant *)&values); - - return new_string; + return String(godot::api->godot_string_format(&_godot_string, (godot_variant *)&values)); } String String::format(Variant values, String placeholder) const { - String new_string; godot_char_string contents = godot::api->godot_string_utf8(&placeholder._godot_string); - new_string._godot_string = godot::api->godot_string_format_with_custom_placeholder(&_godot_string, (godot_variant *)&values, godot::api->godot_char_string_get_data(&contents)); + String new_string(godot::api->godot_string_format_with_custom_placeholder(&_godot_string, (godot_variant *)&values, godot::api->godot_char_string_get_data(&contents))); godot::api->godot_char_string_destroy(&contents); return new_string; } String String::get_base_dir() const { - String new_string; - new_string._godot_string = godot::api->godot_string_get_base_dir(&_godot_string); - - return new_string; + return String(godot::api->godot_string_get_base_dir(&_godot_string)); } String String::get_basename() const { - godot_string new_string = godot::api->godot_string_get_basename(&_godot_string); - return *(String *)&new_string; + return String(godot::api->godot_string_get_basename(&_godot_string)); } String String::get_extension() const { - godot_string new_string = godot::api->godot_string_get_extension(&_godot_string); - return *(String *)&new_string; + return String(godot::api->godot_string_get_extension(&_godot_string)); } String String::get_file() const { - godot_string new_string = godot::api->godot_string_get_file(&_godot_string); - return *(String *)&new_string; + return String(godot::api->godot_string_get_file(&_godot_string)); } int String::hash() const { @@ -314,10 +271,7 @@ int String::hex_to_int() const { } String String::insert(int position, String what) const { - String new_string; - new_string._godot_string = godot::api->godot_string_insert(&_godot_string, position, what._godot_string); - - return new_string; + return String(godot::api->godot_string_insert(&_godot_string, position, what._godot_string)); } bool String::is_abs_path() const { @@ -357,17 +311,11 @@ bool String::is_valid_ip_address() const { } String String::json_escape() const { - String new_string; - new_string._godot_string = godot::api->godot_string_json_escape(&_godot_string); - - return new_string; + return String(godot::api->godot_string_json_escape(&_godot_string)); } String String::left(int position) const { - String new_string; - new_string._godot_string = godot::api->godot_string_left(&_godot_string, position); - - return new_string; + return String(godot::api->godot_string_left(&_godot_string, position)); } bool String::match(String expr) const { @@ -384,10 +332,7 @@ PoolByteArray String::md5_buffer() const { } String String::md5_text() const { - String new_string; - new_string._godot_string = godot::api->godot_string_md5_text(&_godot_string); - - return new_string; + return String(godot::api->godot_string_md5_text(&_godot_string)); } int String::ord_at(int at) const { @@ -395,52 +340,31 @@ int String::ord_at(int at) const { } String String::pad_decimals(int digits) const { - String new_string; - new_string._godot_string = godot::api->godot_string_pad_decimals(&_godot_string, digits); - - return new_string; + return String(godot::api->godot_string_pad_decimals(&_godot_string, digits)); } String String::pad_zeros(int digits) const { - String new_string; - new_string._godot_string = godot::api->godot_string_pad_zeros(&_godot_string, digits); - - return new_string; + return String(godot::api->godot_string_pad_zeros(&_godot_string, digits)); } String String::percent_decode() const { - String new_string; - new_string._godot_string = godot::api->godot_string_percent_decode(&_godot_string); - - return new_string; + return String(godot::api->godot_string_percent_decode(&_godot_string)); } String String::percent_encode() const { - String new_string; - new_string._godot_string = godot::api->godot_string_percent_encode(&_godot_string); - - return new_string; + return String(godot::api->godot_string_percent_encode(&_godot_string)); } String String::plus_file(String file) const { - String new_string; - new_string._godot_string = godot::api->godot_string_plus_file(&_godot_string, &file._godot_string); - - return new_string; + return String(godot::api->godot_string_plus_file(&_godot_string, &file._godot_string)); } String String::replace(String p_key, String p_with) const { - String new_string; - new_string._godot_string = godot::api->godot_string_replace(&_godot_string, p_key._godot_string, p_with._godot_string); - - return new_string; + return String(godot::api->godot_string_replace(&_godot_string, p_key._godot_string, p_with._godot_string)); } String String::replacen(String what, String forwhat) const { - String new_string; - new_string._godot_string = godot::api->godot_string_replacen(&_godot_string, what._godot_string, forwhat._godot_string); - - return new_string; + return String(godot::api->godot_string_replacen(&_godot_string, what._godot_string, forwhat._godot_string)); } int String::rfind(String p_what, int p_from) const { @@ -452,10 +376,7 @@ int String::rfindn(String p_what, int p_from) const { } String String::right(int position) const { - String new_string; - new_string._godot_string = godot::api->godot_string_right(&_godot_string, position); - - return new_string; + return String(godot::api->godot_string_right(&_godot_string, position)); } PoolByteArray String::sha256_buffer() const { @@ -465,10 +386,7 @@ PoolByteArray String::sha256_buffer() const { } String String::sha256_text() const { - String new_string; - new_string._godot_string = godot::api->godot_string_sha256_text(&_godot_string); - - return new_string; + return String(godot::api->godot_string_sha256_text(&_godot_string)); } float String::similarity(String text) const { @@ -494,17 +412,11 @@ PoolRealArray String::split_floats(String divisor, bool allow_empty) const { } String String::strip_edges(bool left, bool right) const { - String new_string; - new_string._godot_string = godot::api->godot_string_strip_edges(&_godot_string, left, right); - - return new_string; + return String(godot::api->godot_string_strip_edges(&_godot_string, left, right)); } String String::substr(int from, int len) const { - String new_string; - new_string._godot_string = godot::api->godot_string_substr(&_godot_string, from, len); - - return new_string; + return String(godot::api->godot_string_substr(&_godot_string, from, len)); } float String::to_float() const { @@ -516,31 +428,19 @@ int64_t String::to_int() const { } String String::to_lower() const { - String new_string; - new_string._godot_string = godot::api->godot_string_to_lower(&_godot_string); - - return new_string; + return String(godot::api->godot_string_to_lower(&_godot_string)); } String String::to_upper() const { - String new_string; - new_string._godot_string = godot::api->godot_string_to_upper(&_godot_string); - - return new_string; + return String(godot::api->godot_string_to_upper(&_godot_string)); } String String::xml_escape() const { - String new_string; - new_string._godot_string = godot::api->godot_string_xml_escape(&_godot_string); - - return new_string; + return String(godot::api->godot_string_xml_escape(&_godot_string)); } String String::xml_unescape() const { - String new_string; - new_string._godot_string = godot::api->godot_string_xml_unescape(&_godot_string); - - return new_string; + return String(godot::api->godot_string_xml_unescape(&_godot_string)); } signed char String::casecmp_to(String p_str) const { From 7d347edb12645e6a7a09539ac1ba10ae1783f2ac Mon Sep 17 00:00:00 2001 From: TerraAr <37753757+TerraAr@users.noreply.github.com> Date: Sat, 15 Aug 2020 15:10:16 -0300 Subject: [PATCH 08/10] Updated operator I've updated the operator that I had created to use the private constructor of the class. --- src/core/String.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/core/String.cpp b/src/core/String.cpp index 35da37f8..72222b7e 100644 --- a/src/core/String.cpp +++ b/src/core/String.cpp @@ -135,8 +135,8 @@ void String::operator+=(const String &s) { } void String::operator+=(const wchar_t c) { - String _to_be_added = c; - _godot_string = godot::api->godot_string_operator_plus(&_godot_string, &_to_be_added._godot_string); + String _to_be_added = String(c); + *this = String(godot::api->godot_string_operator_plus(&_godot_string, &_to_be_added._godot_string)); } bool String::operator<(const String &s) const { From 33f9de16e414ad091fb6d4a6326f5cc435016ef9 Mon Sep 17 00:00:00 2001 From: sheepandshepherd Date: Mon, 21 Oct 2019 18:08:03 +0200 Subject: [PATCH 09/10] Use godot_object_cast_to instead of TagDB to cast engine types --- binding_generator.py | 12 ++++++++++++ include/core/Godot.hpp | 33 +++++++++++++++++++-------------- 2 files changed, 31 insertions(+), 14 deletions(-) diff --git a/binding_generator.py b/binding_generator.py index 13505dbe..2f175920 100644 --- a/binding_generator.py +++ b/binding_generator.py @@ -146,10 +146,14 @@ def generate_class_header(used_classes, c): source.append("\t};") source.append("\tstatic ___method_bindings ___mb;") + source.append("\tstatic void *_detail_class_tag;") source.append("") source.append("public:") source.append("\tstatic void ___init_method_bindings();") + # class id from core engine for casting + source.append("\tinline static size_t ___get_id() { return (size_t)_detail_class_tag; }") + source.append("") @@ -355,11 +359,19 @@ def generate_class_implementation(icalls, used_classes, c): source.append(class_name + "::___method_bindings " + class_name + "::___mb = {};") source.append("") + source.append("void *" + class_name + "::_detail_class_tag = nullptr;") + source.append("") + source.append("void " + class_name + "::___init_method_bindings() {") for method in c["methods"]: source.append("\t___mb.mb_" + method["name"] + " = godot::api->godot_method_bind_get_method(\"" + c["name"] + "\", \"" + method["name"] + "\");") + source.append("\tgodot_string_name class_name;") + source.append("\tgodot::api->godot_string_name_new_data(&class_name, \"" + c["name"] + "\");") + source.append("\t_detail_class_tag = godot::core_1_2_api->godot_get_class_tag(&class_name);") + source.append("\tgodot::api->godot_string_name_destroy(&class_name);") + source.append("}") source.append("") diff --git a/include/core/Godot.hpp b/include/core/Godot.hpp index ddf1e3e0..9d17a2b9 100644 --- a/include/core/Godot.hpp +++ b/include/core/Godot.hpp @@ -100,7 +100,7 @@ public: return godot::detail::create_custom_class_instance(); \ } \ inline static size_t ___get_id() { return typeid(Name).hash_code(); } \ - inline static size_t ___get_base_id() { return typeid(Base).hash_code(); } \ + inline static size_t ___get_base_id() { return Base::___get_id(); } \ inline static const char *___get_base_type_name() { return Base::___get_class_name(); } \ inline static godot::Object *___get_from_variant(godot::Variant a) { \ return (godot::Object *)godot::detail::get_custom_class_instance( \ @@ -513,23 +513,28 @@ T *Object::cast_to(const Object *obj) { if (!obj) return nullptr; - size_t have_tag = (size_t)godot::nativescript_1_1_api->godot_nativescript_get_type_tag(obj->_owner); + if (T::___CLASS_IS_SCRIPT) { + size_t have_tag = (size_t)godot::nativescript_1_1_api->godot_nativescript_get_type_tag(obj->_owner); + if (have_tag) { + if (!godot::_TagDB::is_type_known((size_t)have_tag)) { + have_tag = 0; + } + } - if (have_tag) { - if (!godot::_TagDB::is_type_known((size_t)have_tag)) { - have_tag = 0; + if (!have_tag) { + have_tag = obj->_type_tag; + } + + if (godot::_TagDB::is_type_compatible(T::___get_id(), have_tag)) { + return detail::get_custom_class_instance(obj); + } + } else { + if (godot::core_1_2_api->godot_object_cast_to(obj->_owner, (void *)T::___get_id())) { + return (T *)obj; } } - if (!have_tag) { - have_tag = obj->_type_tag; - } - - if (godot::_TagDB::is_type_compatible(typeid(T).hash_code(), have_tag)) { - return (T::___CLASS_IS_SCRIPT) ? detail::get_custom_class_instance(obj) : (T *)obj; - } else { - return nullptr; - } + return nullptr; } #endif From 63066f2570a90cb1e373562640a67a84f70df642 Mon Sep 17 00:00:00 2001 From: Marc Gilleron Date: Sun, 23 Aug 2020 15:11:57 +0100 Subject: [PATCH 10/10] Replace ".os" with ".o" because it is "Object files" on Linux --- README.md | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/README.md b/README.md index ef8d89ac..a0543484 100644 --- a/README.md +++ b/README.md @@ -221,8 +221,8 @@ Once you've compiled the GDNative C++ bindings (see above), you can compile the ```bash cd SimpleLibrary -clang -fPIC -o src/init.os -c src/init.cpp -g -O3 -std=c++14 -Igodot-cpp/include -Igodot-cpp/include/core -Igodot-cpp/include/gen -Igodot-cpp/godot_headers -clang -o bin/libtest.so -shared src/init.os -Lgodot-cpp/bin -l +clang -fPIC -o src/init.o -c src/init.cpp -g -O3 -std=c++14 -Igodot-cpp/include -Igodot-cpp/include/core -Igodot-cpp/include/gen -Igodot-cpp/godot_headers +clang -o bin/libtest.so -shared src/init.o -Lgodot-cpp/bin -l ``` You'll need to replace `` with the file that was created in [**Compiling the cpp bindings library**](#compiling-the-cpp-bindings-library). @@ -256,8 +256,8 @@ submit a pull request :slightly_smiling_face: ```bash cd SimpleLibrary -aarch64-linux-android29-clang -fPIC -o src/init.os -c src/init.cpp -g -O3 -std=c++14 -Igodot-cpp/include -Igodot-cpp/include/core -Igodot-cpp/include/gen -Igodot-cpp/godot_headers -aarch64-linux-android29-clang -o bin/libtest.so -shared src/init.os -Lgodot-cpp/bin -l +aarch64-linux-android29-clang -fPIC -o src/init.o -c src/init.cpp -g -O3 -std=c++14 -Igodot-cpp/include -Igodot-cpp/include/core -Igodot-cpp/include/gen -Igodot-cpp/godot_headers +aarch64-linux-android29-clang -o bin/libtest.so -shared src/init.o -Lgodot-cpp/bin -l ``` You'll need to replace `` with the file that was created in [**Compiling the cpp bindings library**](#compiling-the-cpp-bindings-library). The command above targets `arm64v8`. To target `armv7`, use `armv7a-linux-androideabi29-clang` instead of `aarch64-linux-android29-clang`.