Преглед на файлове

Fix unexpected Packed Array copying when parameter is typed

cdemirer преди 3 години
родител
ревизия
511e699fee
променени са 1 файла, в които са добавени 6 реда и са изтрити 1 реда
  1. 6 1
      modules/gdscript/gdscript_vm.cpp

+ 6 - 1
modules/gdscript/gdscript_vm.cpp

@@ -488,7 +488,12 @@ Variant GDScriptFunction::call(GDScriptInstance *p_instance, const Variant **p_a
 				memnew_placement(&stack[i + 3], Variant(*p_args[i]));
 				continue;
 			}
-
+			// If types already match, don't call Variant::construct(). Constructors of some types
+			// (e.g. packed arrays) do copies, whereas they pass by reference when inside a Variant.
+			if (argument_types[i].is_type(*p_args[i], false)) {
+				memnew_placement(&stack[i + 3], Variant(*p_args[i]));
+				continue;
+			}
 			if (!argument_types[i].is_type(*p_args[i], true)) {
 				r_err.error = Callable::CallError::CALL_ERROR_INVALID_ARGUMENT;
 				r_err.argument = i;