diff options
author | Gael Guennebaud <g.gael@free.fr> | 2018-07-19 18:33:53 +0200 |
---|---|---|
committer | Gael Guennebaud <g.gael@free.fr> | 2018-07-19 18:33:53 +0200 |
commit | 5e5987996f4a5e9cfda7122c5a0ee09cf4d3874b (patch) | |
tree | 76427e3b08230275a33373e7c925ea4504d3ee04 /Eigen/src/Geometry/Quaternion.h | |
parent | d908afe35f2bf521499abb009602e7029ed65f2d (diff) |
Fix stupid error in Quaternion move ctor
Diffstat (limited to 'Eigen/src/Geometry/Quaternion.h')
-rw-r--r-- | Eigen/src/Geometry/Quaternion.h | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/Eigen/src/Geometry/Quaternion.h b/Eigen/src/Geometry/Quaternion.h index 8b203dc71..faea62f17 100644 --- a/Eigen/src/Geometry/Quaternion.h +++ b/Eigen/src/Geometry/Quaternion.h @@ -280,9 +280,8 @@ public: // We define a copy constructor, which means we don't get an implicit move constructor or assignment operator. /** Default move constructor */ EIGEN_DEVICE_FUNC inline Quaternion(Quaternion&& other) EIGEN_NOEXCEPT_IF(std::is_nothrow_move_constructible<Scalar>::value) - { - m_coeffs(std::move(other.coeffs())); - } + : m_coeffs(std::move(other.coeffs())) + {} /** Default move assignment operator */ EIGEN_DEVICE_FUNC Quaternion& operator=(Quaternion&& other) EIGEN_NOEXCEPT_IF(std::is_nothrow_move_assignable<Scalar>::value) @@ -294,9 +293,8 @@ public: // And now because we declared a constructor, we don't get an implicit copy constructor. Say we want one. /** Default copy constructor */ EIGEN_DEVICE_FUNC Quaternion(const Quaternion& other) - { - m_coeffs = other.coeffs(); - } + : m_coeffs(other.coeffs()) + {} #endif EIGEN_DEVICE_FUNC static Quaternion UnitRandom(); |