1
0
mirror of https://github.com/godotengine/godot.git synced 2025-11-14 13:41:12 +00:00

GDScript: Fix incorrect error message for utility functions

This commit is contained in:
Danil Alexeev
2023-06-30 20:40:02 +03:00
parent a574c0296b
commit 0c2202c56e
13 changed files with 167 additions and 59 deletions

View File

@@ -121,16 +121,27 @@ Variant VariantUtilityFunctions::floor(Variant x, Callable::CallError &r_error)
case Variant::VECTOR2: { case Variant::VECTOR2: {
return VariantInternalAccessor<Vector2>::get(&x).floor(); return VariantInternalAccessor<Vector2>::get(&x).floor();
} break; } break;
case Variant::VECTOR2I: {
return VariantInternalAccessor<Vector2i>::get(&x);
} break;
case Variant::VECTOR3: { case Variant::VECTOR3: {
return VariantInternalAccessor<Vector3>::get(&x).floor(); return VariantInternalAccessor<Vector3>::get(&x).floor();
} break; } break;
case Variant::VECTOR3I: {
return VariantInternalAccessor<Vector3i>::get(&x);
} break;
case Variant::VECTOR4: { case Variant::VECTOR4: {
return VariantInternalAccessor<Vector4>::get(&x).floor(); return VariantInternalAccessor<Vector4>::get(&x).floor();
} break; } break;
case Variant::VECTOR4I: {
return VariantInternalAccessor<Vector4i>::get(&x);
} break;
default: { default: {
r_error.error = Callable::CallError::CALL_ERROR_INVALID_METHOD; r_error.error = Callable::CallError::CALL_ERROR_INVALID_ARGUMENT;
return Variant(); r_error.argument = 0;
} r_error.expected = Variant::NIL;
return R"(Argument "x" must be "int", "float", "Vector2", "Vector2i", "Vector3", "Vector3i", "Vector4", or "Vector4i".)";
} break;
} }
} }
@@ -154,16 +165,27 @@ Variant VariantUtilityFunctions::ceil(Variant x, Callable::CallError &r_error) {
case Variant::VECTOR2: { case Variant::VECTOR2: {
return VariantInternalAccessor<Vector2>::get(&x).ceil(); return VariantInternalAccessor<Vector2>::get(&x).ceil();
} break; } break;
case Variant::VECTOR2I: {
return VariantInternalAccessor<Vector2i>::get(&x);
} break;
case Variant::VECTOR3: { case Variant::VECTOR3: {
return VariantInternalAccessor<Vector3>::get(&x).ceil(); return VariantInternalAccessor<Vector3>::get(&x).ceil();
} break; } break;
case Variant::VECTOR3I: {
return VariantInternalAccessor<Vector3i>::get(&x);
} break;
case Variant::VECTOR4: { case Variant::VECTOR4: {
return VariantInternalAccessor<Vector4>::get(&x).ceil(); return VariantInternalAccessor<Vector4>::get(&x).ceil();
} break; } break;
case Variant::VECTOR4I: {
return VariantInternalAccessor<Vector4i>::get(&x);
} break;
default: { default: {
r_error.error = Callable::CallError::CALL_ERROR_INVALID_METHOD; r_error.error = Callable::CallError::CALL_ERROR_INVALID_ARGUMENT;
return Variant(); r_error.argument = 0;
} r_error.expected = Variant::NIL;
return R"(Argument "x" must be "int", "float", "Vector2", "Vector2i", "Vector3", "Vector3i", "Vector4", or "Vector4i".)";
} break;
} }
} }
@@ -187,16 +209,27 @@ Variant VariantUtilityFunctions::round(Variant x, Callable::CallError &r_error)
case Variant::VECTOR2: { case Variant::VECTOR2: {
return VariantInternalAccessor<Vector2>::get(&x).round(); return VariantInternalAccessor<Vector2>::get(&x).round();
} break; } break;
case Variant::VECTOR2I: {
return VariantInternalAccessor<Vector2i>::get(&x);
} break;
case Variant::VECTOR3: { case Variant::VECTOR3: {
return VariantInternalAccessor<Vector3>::get(&x).round(); return VariantInternalAccessor<Vector3>::get(&x).round();
} break; } break;
case Variant::VECTOR3I: {
return VariantInternalAccessor<Vector3i>::get(&x);
} break;
case Variant::VECTOR4: { case Variant::VECTOR4: {
return VariantInternalAccessor<Vector4>::get(&x).round(); return VariantInternalAccessor<Vector4>::get(&x).round();
} break; } break;
case Variant::VECTOR4I: {
return VariantInternalAccessor<Vector4i>::get(&x);
} break;
default: { default: {
r_error.error = Callable::CallError::CALL_ERROR_INVALID_METHOD; r_error.error = Callable::CallError::CALL_ERROR_INVALID_ARGUMENT;
return Variant(); r_error.argument = 0;
} r_error.expected = Variant::NIL;
return R"(Argument "x" must be "int", "float", "Vector2", "Vector2i", "Vector3", "Vector3i", "Vector4", or "Vector4i".)";
} break;
} }
} }
@@ -236,9 +269,11 @@ Variant VariantUtilityFunctions::abs(const Variant &x, Callable::CallError &r_er
return VariantInternalAccessor<Vector4i>::get(&x).abs(); return VariantInternalAccessor<Vector4i>::get(&x).abs();
} break; } break;
default: { default: {
r_error.error = Callable::CallError::CALL_ERROR_INVALID_METHOD; r_error.error = Callable::CallError::CALL_ERROR_INVALID_ARGUMENT;
return Variant(); r_error.argument = 0;
} r_error.expected = Variant::NIL;
return R"(Argument "x" must be "int", "float", "Vector2", "Vector2i", "Vector3", "Vector3i", "Vector4", or "Vector4i".)";
} break;
} }
} }
@@ -278,9 +313,11 @@ Variant VariantUtilityFunctions::sign(const Variant &x, Callable::CallError &r_e
return VariantInternalAccessor<Vector4i>::get(&x).sign(); return VariantInternalAccessor<Vector4i>::get(&x).sign();
} break; } break;
default: { default: {
r_error.error = Callable::CallError::CALL_ERROR_INVALID_METHOD; r_error.error = Callable::CallError::CALL_ERROR_INVALID_ARGUMENT;
return Variant(); r_error.argument = 0;
} r_error.expected = Variant::NIL;
return R"(Argument "x" must be "int", "float", "Vector2", "Vector2i", "Vector3", "Vector3i", "Vector4", or "Vector4i".)";
} break;
} }
} }
@@ -333,14 +370,40 @@ int VariantUtilityFunctions::step_decimals(float step) {
} }
Variant VariantUtilityFunctions::snapped(const Variant &x, const Variant &step, Callable::CallError &r_error) { Variant VariantUtilityFunctions::snapped(const Variant &x, const Variant &step, Callable::CallError &r_error) {
r_error.error = Callable::CallError::CALL_OK; switch (x.get_type()) {
if (x.get_type() != step.get_type() && !((x.get_type() == Variant::INT && step.get_type() == Variant::FLOAT) || (x.get_type() == Variant::FLOAT && step.get_type() == Variant::INT))) { case Variant::INT:
r_error.error = Callable::CallError::CALL_ERROR_INVALID_ARGUMENT; case Variant::FLOAT:
r_error.argument = 1; case Variant::VECTOR2:
r_error.expected = x.get_type(); case Variant::VECTOR2I:
return Variant(); case Variant::VECTOR3:
case Variant::VECTOR3I:
case Variant::VECTOR4:
case Variant::VECTOR4I:
break;
default:
r_error.error = Callable::CallError::CALL_ERROR_INVALID_ARGUMENT;
r_error.argument = 0;
r_error.expected = Variant::NIL;
return R"(Argument "x" must be "int", "float", "Vector2", "Vector2i", "Vector3", "Vector3i", "Vector4", or "Vector4i".)";
} }
if (x.get_type() != step.get_type()) {
if (x.get_type() == Variant::INT || x.get_type() == Variant::FLOAT) {
if (step.get_type() != Variant::INT && step.get_type() != Variant::FLOAT) {
r_error.error = Callable::CallError::CALL_ERROR_INVALID_ARGUMENT;
r_error.argument = 1;
r_error.expected = Variant::NIL;
return R"(Argument "step" must be "int" or "float".)";
}
} else {
r_error.error = Callable::CallError::CALL_ERROR_INVALID_ARGUMENT;
r_error.argument = 1;
r_error.expected = x.get_type();
return Variant();
}
}
r_error.error = Callable::CallError::CALL_OK;
switch (step.get_type()) { switch (step.get_type()) {
case Variant::INT: { case Variant::INT: {
return snappedi(x, VariantInternalAccessor<int64_t>::get(&step)); return snappedi(x, VariantInternalAccessor<int64_t>::get(&step));
@@ -367,9 +430,8 @@ Variant VariantUtilityFunctions::snapped(const Variant &x, const Variant &step,
return VariantInternalAccessor<Vector4i>::get(&x).snapped(VariantInternalAccessor<Vector4i>::get(&step)); return VariantInternalAccessor<Vector4i>::get(&x).snapped(VariantInternalAccessor<Vector4i>::get(&step));
} break; } break;
default: { default: {
r_error.error = Callable::CallError::CALL_ERROR_INVALID_METHOD; return Variant(); // Already handled.
return Variant(); } break;
}
} }
} }
@@ -382,7 +444,23 @@ int64_t VariantUtilityFunctions::snappedi(double x, int64_t step) {
} }
Variant VariantUtilityFunctions::lerp(const Variant &from, const Variant &to, double weight, Callable::CallError &r_error) { Variant VariantUtilityFunctions::lerp(const Variant &from, const Variant &to, double weight, Callable::CallError &r_error) {
r_error.error = Callable::CallError::CALL_OK; switch (from.get_type()) {
case Variant::INT:
case Variant::FLOAT:
case Variant::VECTOR2:
case Variant::VECTOR3:
case Variant::VECTOR4:
case Variant::QUATERNION:
case Variant::BASIS:
case Variant::COLOR:
break;
default:
r_error.error = Callable::CallError::CALL_ERROR_INVALID_ARGUMENT;
r_error.argument = 0;
r_error.expected = Variant::NIL;
return R"(Argument "from" must be "int", "float", "Vector2", "Vector3", "Vector4", "Quaternion", "Basis, or "Color".)";
}
if (from.get_type() != to.get_type()) { if (from.get_type() != to.get_type()) {
r_error.error = Callable::CallError::CALL_ERROR_INVALID_ARGUMENT; r_error.error = Callable::CallError::CALL_ERROR_INVALID_ARGUMENT;
r_error.argument = 1; r_error.argument = 1;
@@ -390,6 +468,7 @@ Variant VariantUtilityFunctions::lerp(const Variant &from, const Variant &to, do
return Variant(); return Variant();
} }
r_error.error = Callable::CallError::CALL_OK;
switch (from.get_type()) { switch (from.get_type()) {
case Variant::INT: { case Variant::INT: {
return lerpf(VariantInternalAccessor<int64_t>::get(&from), to, weight); return lerpf(VariantInternalAccessor<int64_t>::get(&from), to, weight);
@@ -416,9 +495,8 @@ Variant VariantUtilityFunctions::lerp(const Variant &from, const Variant &to, do
return VariantInternalAccessor<Color>::get(&from).lerp(VariantInternalAccessor<Color>::get(&to), weight); return VariantInternalAccessor<Color>::get(&from).lerp(VariantInternalAccessor<Color>::get(&to), weight);
} break; } break;
default: { default: {
r_error.error = Callable::CallError::CALL_ERROR_INVALID_METHOD; return Variant(); // Already handled.
return Variant(); } break;
}
} }
} }

View File

@@ -196,7 +196,7 @@
<return type="Variant" /> <return type="Variant" />
<param index="0" name="x" type="Variant" /> <param index="0" name="x" type="Variant" />
<description> <description>
Rounds [param x] upward (towards positive infinity), returning the smallest whole number that is not less than [param x]. Supported types: [int], [float], [Vector2], [Vector3], [Vector4]. Rounds [param x] upward (towards positive infinity), returning the smallest whole number that is not less than [param x]. Supported types: [int], [float], [Vector2], [Vector2i], [Vector3], [Vector3i], [Vector4], [Vector4i].
[codeblock] [codeblock]
var i = ceil(1.45) # i is 2.0 var i = ceil(1.45) # i is 2.0
i = ceil(1.001) # i is 2.0 i = ceil(1.001) # i is 2.0
@@ -421,7 +421,7 @@
<return type="Variant" /> <return type="Variant" />
<param index="0" name="x" type="Variant" /> <param index="0" name="x" type="Variant" />
<description> <description>
Rounds [param x] downward (towards negative infinity), returning the largest whole number that is not more than [param x]. Supported types: [int], [float], [Vector2], [Vector3], [Vector4]. Rounds [param x] downward (towards negative infinity), returning the largest whole number that is not more than [param x]. Supported types: [int], [float], [Vector2], [Vector2i], [Vector3], [Vector3i], [Vector4], [Vector4i].
[codeblock] [codeblock]
var a = floor(2.99) # a is 2.0 var a = floor(2.99) # a is 2.0
a = floor(-2.99) # a is -3.0 a = floor(-2.99) # a is -3.0
@@ -1133,7 +1133,7 @@
<return type="Variant" /> <return type="Variant" />
<param index="0" name="x" type="Variant" /> <param index="0" name="x" type="Variant" />
<description> <description>
Rounds [param x] to the nearest whole number, with halfway cases rounded away from 0. Supported types: [int], [float], [Vector2], [Vector3], [Vector4]. Rounds [param x] to the nearest whole number, with halfway cases rounded away from 0. Supported types: [int], [float], [Vector2], [Vector2i], [Vector3], [Vector3i], [Vector4], [Vector4i].
[codeblock] [codeblock]
round(2.4) # Returns 2 round(2.4) # Returns 2
round(2.5) # Returns 3 round(2.5) # Returns 3

View File

@@ -3134,12 +3134,16 @@ void GDScriptAnalyzer::reduce_call(GDScriptParser::CallNode *p_call, bool p_is_a
GDScriptUtilityFunctions::get_function(function_name)(&value, (const Variant **)args.ptr(), args.size(), err); GDScriptUtilityFunctions::get_function(function_name)(&value, (const Variant **)args.ptr(), args.size(), err);
switch (err.error) { switch (err.error) {
case Callable::CallError::CALL_ERROR_INVALID_ARGUMENT: { case Callable::CallError::CALL_ERROR_INVALID_ARGUMENT:
PropertyInfo wrong_arg = function_info.arguments[err.argument]; if (value.get_type() == Variant::STRING && !value.operator String().is_empty()) {
push_error(vformat(R"*(Invalid argument for "%s()" function: argument %d should be "%s" but is "%s".)*", function_name, err.argument + 1, push_error(vformat(R"*(Invalid argument for "%s()" function: %s)*", function_name, value), p_call->arguments[err.argument]);
type_from_property(wrong_arg, true).to_string(), p_call->arguments[err.argument]->get_datatype().to_string()), } else {
p_call->arguments[err.argument]); // Do not use `type_from_property()` for expected type, since utility functions use their own checks.
} break; push_error(vformat(R"*(Invalid argument for "%s()" function: argument %d should be "%s" but is "%s".)*", function_name, err.argument + 1,
Variant::get_type_name((Variant::Type)err.expected), p_call->arguments[err.argument]->get_datatype().to_string()),
p_call->arguments[err.argument]);
}
break;
case Callable::CallError::CALL_ERROR_INVALID_METHOD: case Callable::CallError::CALL_ERROR_INVALID_METHOD:
push_error(vformat(R"(Invalid call for function "%s".)", function_name), p_call); push_error(vformat(R"(Invalid call for function "%s".)", function_name), p_call);
break; break;
@@ -3181,18 +3185,16 @@ void GDScriptAnalyzer::reduce_call(GDScriptParser::CallNode *p_call, bool p_is_a
Variant::call_utility_function(function_name, &value, (const Variant **)args.ptr(), args.size(), err); Variant::call_utility_function(function_name, &value, (const Variant **)args.ptr(), args.size(), err);
switch (err.error) { switch (err.error) {
case Callable::CallError::CALL_ERROR_INVALID_ARGUMENT: { case Callable::CallError::CALL_ERROR_INVALID_ARGUMENT:
String expected_type_name; if (value.get_type() == Variant::STRING && !value.operator String().is_empty()) {
if (err.argument < function_info.arguments.size()) { push_error(vformat(R"*(Invalid argument for "%s()" function: %s)*", function_name, value), p_call->arguments[err.argument]);
expected_type_name = type_from_property(function_info.arguments[err.argument], true).to_string();
} else { } else {
expected_type_name = Variant::get_type_name((Variant::Type)err.expected); // Do not use `type_from_property()` for expected type, since utility functions use their own checks.
push_error(vformat(R"*(Invalid argument for "%s()" function: argument %d should be "%s" but is "%s".)*", function_name, err.argument + 1,
Variant::get_type_name((Variant::Type)err.expected), p_call->arguments[err.argument]->get_datatype().to_string()),
p_call->arguments[err.argument]);
} }
break;
push_error(vformat(R"*(Invalid argument for "%s()" function: argument %d should be "%s" but is "%s".)*", function_name, err.argument + 1,
expected_type_name, p_call->arguments[err.argument]->get_datatype().to_string()),
p_call->arguments[err.argument]);
} break;
case Callable::CallError::CALL_ERROR_INVALID_METHOD: case Callable::CallError::CALL_ERROR_INVALID_METHOD:
push_error(vformat(R"(Invalid call for function "%s".)", function_name), p_call); push_error(vformat(R"(Invalid call for function "%s".)", function_name), p_call);
break; break;

View File

@@ -97,6 +97,9 @@ struct GDScriptUtilityFunctionsDefinitions {
} else { } else {
Variant::construct(Variant::Type(type), *r_ret, p_args, 1, r_error); Variant::construct(Variant::Type(type), *r_ret, p_args, 1, r_error);
if (r_error.error != Callable::CallError::CALL_OK) {
*r_ret = vformat(RTR(R"(Cannot convert "%s" to "%s".)"), Variant::get_type_name(p_args[0]->get_type()), Variant::get_type_name(Variant::Type(type)));
}
} }
} }
#endif // DISABLE_DEPRECATED #endif // DISABLE_DEPRECATED
@@ -130,8 +133,8 @@ struct GDScriptUtilityFunctionsDefinitions {
} }
Error err = arr.resize(count); Error err = arr.resize(count);
if (err != OK) { if (err != OK) {
*r_ret = RTR("Cannot resize array.");
r_error.error = Callable::CallError::CALL_ERROR_INVALID_METHOD; r_error.error = Callable::CallError::CALL_ERROR_INVALID_METHOD;
*r_ret = Variant();
return; return;
} }
@@ -155,8 +158,8 @@ struct GDScriptUtilityFunctionsDefinitions {
} }
Error err = arr.resize(to - from); Error err = arr.resize(to - from);
if (err != OK) { if (err != OK) {
*r_ret = RTR("Cannot resize array.");
r_error.error = Callable::CallError::CALL_ERROR_INVALID_METHOD; r_error.error = Callable::CallError::CALL_ERROR_INVALID_METHOD;
*r_ret = Variant();
return; return;
} }
for (int i = from; i < to; i++) { for (int i = from; i < to; i++) {
@@ -199,8 +202,8 @@ struct GDScriptUtilityFunctionsDefinitions {
Error err = arr.resize(count); Error err = arr.resize(count);
if (err != OK) { if (err != OK) {
*r_ret = RTR("Cannot resize array.");
r_error.error = Callable::CallError::CALL_ERROR_INVALID_METHOD; r_error.error = Callable::CallError::CALL_ERROR_INVALID_METHOD;
*r_ret = Variant();
return; return;
} }
@@ -370,7 +373,7 @@ struct GDScriptUtilityFunctionsDefinitions {
*r_ret = gdscr->_new(nullptr, -1 /*skip initializer*/, r_error); *r_ret = gdscr->_new(nullptr, -1 /*skip initializer*/, r_error);
if (r_error.error != Callable::CallError::CALL_OK) { if (r_error.error != Callable::CallError::CALL_OK) {
*r_ret = Variant(); *r_ret = RTR("Cannot instantiate GDScript class.");
return; return;
} }

View File

@@ -2067,11 +2067,11 @@ Variant GDScriptFunction::call(GDScriptInstance *p_instance, const Variant **p_a
#ifdef DEBUG_ENABLED #ifdef DEBUG_ENABLED
if (err.error != Callable::CallError::CALL_OK) { if (err.error != Callable::CallError::CALL_OK) {
String methodstr = function; String methodstr = function;
if (dst->get_type() == Variant::STRING) { if (dst->get_type() == Variant::STRING && !dst->operator String().is_empty()) {
// Call provided error string. // Call provided error string.
err_text = "Error calling utility function '" + methodstr + "': " + String(*dst); err_text = vformat(R"*(Error calling utility function "%s()": %s)*", methodstr, *dst);
} else { } else {
err_text = _get_call_error(err, "utility function '" + methodstr + "'", (const Variant **)argptrs); err_text = _get_call_error(err, vformat(R"*(utility function "%s()")*", methodstr), (const Variant **)argptrs);
} }
OPCODE_BREAK; OPCODE_BREAK;
} }
@@ -2123,13 +2123,12 @@ Variant GDScriptFunction::call(GDScriptInstance *p_instance, const Variant **p_a
#ifdef DEBUG_ENABLED #ifdef DEBUG_ENABLED
if (err.error != Callable::CallError::CALL_OK) { if (err.error != Callable::CallError::CALL_OK) {
// TODO: Add this information in debug. String methodstr = gds_utilities_names[_code_ptr[ip + 2]];
String methodstr = "<unknown function>"; if (dst->get_type() == Variant::STRING && !dst->operator String().is_empty()) {
if (dst->get_type() == Variant::STRING) {
// Call provided error string. // Call provided error string.
err_text = "Error calling GDScript utility function '" + methodstr + "': " + String(*dst); err_text = vformat(R"*(Error calling GDScript utility function "%s()": %s)*", methodstr, *dst);
} else { } else {
err_text = _get_call_error(err, "GDScript utility function '" + methodstr + "'", (const Variant **)argptrs); err_text = _get_call_error(err, vformat(R"*(GDScript utility function "%s()")*", methodstr), (const Variant **)argptrs);
} }
OPCODE_BREAK; OPCODE_BREAK;
} }

View File

@@ -0,0 +1,2 @@
func test():
print(len(Color())) # GDScript utility function.

View File

@@ -0,0 +1,2 @@
GDTEST_ANALYZER_ERROR
Invalid argument for "len()" function: Value of type 'Color' can't provide a length.

View File

@@ -0,0 +1,2 @@
func test():
print(floor(Color())) # Built-in utility function.

View File

@@ -0,0 +1,2 @@
GDTEST_ANALYZER_ERROR
Invalid argument for "floor()" function: Argument "x" must be "int", "float", "Vector2", "Vector2i", "Vector3", "Vector3i", "Vector4", or "Vector4i".

View File

@@ -0,0 +1,3 @@
func test():
var x = Color()
print(len(x)) # GDScript utility function.

View File

@@ -0,0 +1,6 @@
GDTEST_RUNTIME_ERROR
>> SCRIPT ERROR
>> on function: test()
>> runtime/errors/gd_utility_function_wrong_arg.gd
>> 3
>> Error calling GDScript utility function "len()": Value of type 'Color' can't provide a length.

View File

@@ -0,0 +1,3 @@
func test():
var x = Color()
print(floor(x)) # Built-in utility function.

View File

@@ -0,0 +1,6 @@
GDTEST_RUNTIME_ERROR
>> SCRIPT ERROR
>> on function: test()
>> runtime/errors/utility_function_wrong_arg.gd
>> 3
>> Error calling utility function "floor()": Argument "x" must be "int", "float", "Vector2", "Vector2i", "Vector3", "Vector3i", "Vector4", or "Vector4i".