From 21bb5b9c0e9a411e0210ce0e76529963e616a0c4 Mon Sep 17 00:00:00 2001 From: George Marques Date: Wed, 26 Apr 2023 10:57:22 -0300 Subject: GDScript: Don't fail when freed object is return This is check is a bit too eager. The user should be able to handle the return value even if it's a freed object. (cherry picked from commit abbdf806435aad401f2a7b6ce09838c8e4d5cbcb) --- .../scripts/runtime/features/getter_with_freed_object.gd | 15 +++++++++++++++ .../scripts/runtime/features/getter_with_freed_object.out | 2 ++ 2 files changed, 17 insertions(+) create mode 100644 modules/gdscript/tests/scripts/runtime/features/getter_with_freed_object.gd create mode 100644 modules/gdscript/tests/scripts/runtime/features/getter_with_freed_object.out (limited to 'modules/gdscript/tests') diff --git a/modules/gdscript/tests/scripts/runtime/features/getter_with_freed_object.gd b/modules/gdscript/tests/scripts/runtime/features/getter_with_freed_object.gd new file mode 100644 index 0000000000..a2d09bf7d3 --- /dev/null +++ b/modules/gdscript/tests/scripts/runtime/features/getter_with_freed_object.gd @@ -0,0 +1,15 @@ +# https://github.com/godotengine/godot/issues/68184 + +var node: Node: + get: + return node + set(n): + node = n + + +func test(): + node = Node.new() + node.free() + + if !is_instance_valid(node): + print("It is freed") diff --git a/modules/gdscript/tests/scripts/runtime/features/getter_with_freed_object.out b/modules/gdscript/tests/scripts/runtime/features/getter_with_freed_object.out new file mode 100644 index 0000000000..b380f593d9 --- /dev/null +++ b/modules/gdscript/tests/scripts/runtime/features/getter_with_freed_object.out @@ -0,0 +1,2 @@ +GDTEST_OK +It is freed -- cgit v1.2.3