Merge pull request #105546 from Splizard/fix_gdextension_tostring
Fix GDExtension `Object/Node::to_string` to check `is_valid` before returning the result
This commit is contained in:
@ -978,8 +978,10 @@ String Object::to_string() {
|
|||||||
String ret;
|
String ret;
|
||||||
GDExtensionBool is_valid;
|
GDExtensionBool is_valid;
|
||||||
_extension->to_string(_extension_instance, &is_valid, &ret);
|
_extension->to_string(_extension_instance, &is_valid, &ret);
|
||||||
|
if (is_valid) {
|
||||||
return ret;
|
return ret;
|
||||||
}
|
}
|
||||||
|
}
|
||||||
return "<" + get_class() + "#" + itos(get_instance_id()) + ">";
|
return "<" + get_class() + "#" + itos(get_instance_id()) + ">";
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
@ -2864,8 +2864,10 @@ String Node::to_string() {
|
|||||||
String ret;
|
String ret;
|
||||||
GDExtensionBool is_valid;
|
GDExtensionBool is_valid;
|
||||||
_get_extension()->to_string(_get_extension_instance(), &is_valid, &ret);
|
_get_extension()->to_string(_get_extension_instance(), &is_valid, &ret);
|
||||||
|
if (is_valid) {
|
||||||
return ret;
|
return ret;
|
||||||
}
|
}
|
||||||
|
}
|
||||||
return (get_name() ? String(get_name()) + ":" : "") + Object::to_string();
|
return (get_name() ? String(get_name()) + ":" : "") + Object::to_string();
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user