diff options
author | Ferenc Arn <tagcup@yahoo.com> | 2017-03-23 12:27:00 -0500 |
---|---|---|
committer | Ferenc Arn <tagcup@yahoo.com> | 2017-03-23 12:27:00 -0500 |
commit | 6bb9b58b09aee7fc543c03844ce51b62c838dadd (patch) | |
tree | dcb9a80383698af2b9d741f151a03f5a2b739987 /core/math/matrix3.cpp | |
parent | c37fad650f92845a6f59740fea2fea1b46f56db1 (diff) | |
download | redot-engine-6bb9b58b09aee7fc543c03844ce51b62c838dadd.tar.gz |
Explicitly documented that Transform.basis is not necessarily an orthogonal matrix.
Also added a check that in axis-angle rotations, axis is a normalized vector, and modified the docs accordingly.
Fixes #8113.
Diffstat (limited to 'core/math/matrix3.cpp')
-rw-r--r-- | core/math/matrix3.cpp | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/core/math/matrix3.cpp b/core/math/matrix3.cpp index 5f73d91ef3..f154552a3e 100644 --- a/core/math/matrix3.cpp +++ b/core/math/matrix3.cpp @@ -575,6 +575,8 @@ Basis::Basis(const Quat &p_quat) { Basis::Basis(const Vector3 &p_axis, real_t p_phi) { // Rotation matrix from axis and angle, see https://en.wikipedia.org/wiki/Rotation_matrix#Rotation_matrix_from_axis_and_angle + ERR_FAIL_COND(p_axis.is_normalized() == false); + Vector3 axis_sq(p_axis.x * p_axis.x, p_axis.y * p_axis.y, p_axis.z * p_axis.z); real_t cosine = Math::cos(p_phi); |