diff options
| author | Stanislaw Halik <sthalik@misaki.pl> | 2017-04-14 07:11:26 +0200 | 
|---|---|---|
| committer | Stanislaw Halik <sthalik@misaki.pl> | 2017-04-14 07:11:26 +0200 | 
| commit | f218f1ea6258e18af38aee6469c70c787ff6de01 (patch) | |
| tree | 181d11e100109d37b05479fe1c4d16e1345d4dfa | |
| parent | acb12232452d6a4be368702eedaa1ba871653eef (diff) | |
tracker/steamvr: use double promotion for all arithmetic
The computation order isn't defined so parts might only get
promoted to double after arithmetic. Promote all parts manually
before arithmetic.
| -rw-r--r-- | tracker-steamvr/steamvr.cpp | 8 | 
1 files changed, 5 insertions, 3 deletions
| diff --git a/tracker-steamvr/steamvr.cpp b/tracker-steamvr/steamvr.cpp index 8998149c..d32c9d44 100644 --- a/tracker-steamvr/steamvr.cpp +++ b/tracker-steamvr/steamvr.cpp @@ -275,9 +275,11 @@ void steamvr::matrix_to_euler(double& yaw, double& pitch, double& roll, const vr      using std::atan2;      using std::sqrt; -    yaw = atan2((double)-result.m[2][0], sqrt(double(result.m[2][1]) * result.m[2][1] + result.m[2][2] * result.m[2][2])); -    pitch = atan2((double)result.m[2][1], (double)result.m[2][2]); -    roll = atan2((double)result.m[1][0], (double)result.m[0][0]); +    using d = double; + +    yaw = atan2(d(-result.m[2][0]), sqrt(d(result.m[2][1]) * d(result.m[2][1]) + d(result.m[2][2]) * d(result.m[2][2]))); +    pitch = atan2(d(result.m[2][1]), d(result.m[2][2])); +    roll = atan2(d(result.m[1][0]), d(result.m[0][0]));      // TODO: gimbal lock avoidance?  } | 
