diff options
author | Marc <marc.gilleron@gmail.com> | 2021-01-31 20:15:44 +0000 |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-01-31 20:15:44 +0000 |
commit | e8de1fa2a5e2974c273fde35bc5ecac375ae98ec (patch) | |
tree | 286a8e2e25c18fe11f346835b3021afae2e9b77f /src/core/NodePath.cpp | |
parent | 05ba977cc60653952b73dc03498ebc7a93cef120 (diff) | |
parent | b400dba87534640eeddbcdb6b319335a6a7639d8 (diff) | |
download | redot-cpp-e8de1fa2a5e2974c273fde35bc5ecac375ae98ec.tar.gz |
Merge pull request #490 from Zylann/container_leaks
Fix container and string leaks
Diffstat (limited to 'src/core/NodePath.cpp')
-rw-r--r-- | src/core/NodePath.cpp | 12 |
1 files changed, 5 insertions, 7 deletions
diff --git a/src/core/NodePath.cpp b/src/core/NodePath.cpp index 37ab2a3..bc069b7 100644 --- a/src/core/NodePath.cpp +++ b/src/core/NodePath.cpp @@ -27,8 +27,7 @@ NodePath::NodePath(const char *contents) { String NodePath::get_name(const int idx) const { godot_string str = godot::api->godot_node_path_get_name(&_node_path, idx); - - return *(String *)&str; + return String(str); } int NodePath::get_name_count() const { @@ -37,7 +36,7 @@ int NodePath::get_name_count() const { String NodePath::get_subname(const int idx) const { godot_string str = godot::api->godot_node_path_get_subname(&_node_path, idx); - return *(String *)&str; + return String(str); } int NodePath::get_subname_count() const { @@ -54,17 +53,16 @@ bool NodePath::is_empty() const { NodePath NodePath::get_as_property_path() const { godot_node_path path = godot::core_1_1_api->godot_node_path_get_as_property_path(&_node_path); - return *(NodePath *)&path; + return NodePath(path); } String NodePath::get_concatenated_subnames() const { godot_string str = godot::api->godot_node_path_get_concatenated_subnames(&_node_path); - return *(String *)&str; + return String(str); } NodePath::operator String() const { godot_string str = godot::api->godot_node_path_as_string(&_node_path); - - return *(String *)&str; + return String(str); } bool NodePath::operator==(const NodePath &other) { |