You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Reproducible in v4.3.stable.official [77dcf97] or any 4.x
System information
Windows 11
Issue description
lerp accepts any Variant as a weight, altough it only supports float. Giving other values causes the function to fail silently. This problem was mentioned in #64332 as well, but I think it is a separate issue.
The function is registered with a macro FUNCBINDVR3, which doesn't offer flexibility around the argument types. Possible solutions: inline the contents of the macro and write correct get_argument_type(int p_arg) function. Or modify the macro to relay the correct argument types.
Steps to reproduce
Write
lerp(0.0, 10.0, "hello")
or just see the reference...
Minimal reproduction project (MRP)
N/A
The text was updated successfully, but these errors were encountered:
It generates the designed code, but doesn't consider mixed argument specificity, and the macros ensure things aren't accentially missed, so inlining it would be potentially error prone
But this would need to be tested and discussed more generally, especially since it still would break compatibility, and we can't fix compatibility in these methods so that will mess with extensions
Tested versions
Reproducible in v4.3.stable.official [77dcf97] or any 4.x
System information
Windows 11
Issue description
lerp
accepts any Variant as a weight, altough it only supports float. Giving other values causes the function to fail silently. This problem was mentioned in #64332 as well, but I think it is a separate issue.The function is registered with a macro
FUNCBINDVR3
, which doesn't offer flexibility around the argument types. Possible solutions: inline the contents of the macro and write correctget_argument_type(int p_arg)
function. Or modify the macro to relay the correct argument types.Steps to reproduce
Write
or just see the reference...
Minimal reproduction project (MRP)
N/A
The text was updated successfully, but these errors were encountered: