Default constructor now default

2015-07-12 18:56:45 -07:00 · 2015-07-12 18:56:45 -07:00 · 9c525c2e7f
parent 539401664b
commit 9c525c2e7f
1 changed files with 2 additions and 2 deletions
--- a/gtsam/base/Manifold.h
+++ b/gtsam/base/Manifold.h
@ -187,8 +187,8 @@ public:
  typedef Eigen::Matrix<double, dimension, 1> TangentVector;
  typedef OptionalJacobian<dimension, dimension> ChartJacobian;

-  /// Default constructor yields identity
-  ProductManifold():std::pair<M1,M2>(traits<M1>::Identity(),traits<M2>::Identity()) {}
+  /// Default constructor needs default constructors to be defined
+  ProductManifold():std::pair<M1,M2>(M1(),M2()) {}

  // Construct from two original manifold values
  ProductManifold(const M1& m1, const M2& m2):std::pair<M1,M2>(m1,m2) {}