Fix stupid error in Quaternion move ctor

This commit is contained in:
Gael Guennebaud 2018-07-19 18:33:53 +02:00
parent d908afe35f
commit 5e5987996f

View File

@ -280,9 +280,8 @@ public:
// We define a copy constructor, which means we don't get an implicit move constructor or assignment operator. // We define a copy constructor, which means we don't get an implicit move constructor or assignment operator.
/** Default move constructor */ /** Default move constructor */
EIGEN_DEVICE_FUNC inline Quaternion(Quaternion&& other) EIGEN_NOEXCEPT_IF(std::is_nothrow_move_constructible<Scalar>::value) 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 */ /** Default move assignment operator */
EIGEN_DEVICE_FUNC Quaternion& operator=(Quaternion&& other) EIGEN_NOEXCEPT_IF(std::is_nothrow_move_assignable<Scalar>::value) 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. // And now because we declared a constructor, we don't get an implicit copy constructor. Say we want one.
/** Default copy constructor */ /** Default copy constructor */
EIGEN_DEVICE_FUNC Quaternion(const Quaternion& other) EIGEN_DEVICE_FUNC Quaternion(const Quaternion& other)
{ : m_coeffs(other.coeffs())
m_coeffs = other.coeffs(); {}
}
#endif #endif
EIGEN_DEVICE_FUNC static Quaternion UnitRandom(); EIGEN_DEVICE_FUNC static Quaternion UnitRandom();