Merge pull request #130 from SimonRohou/codac2_dev

[mat] corrected iterator

Author: SimonRohou

python/src/core/matrices/codac2_py_VectorBase.h

@@ -101,8 +101,7 @@ void export_VectorBase(py::module& m, py::class_<S>& pyclass)
 
     .def("__iter__", [](const S &x)
         {
-          auto x_ = x._e.reshaped();
-          return py::make_iterator(x_.begin(), x_.end());
+          return py::make_iterator(x._e.begin(), x._e.end());
         },
       py::keep_alive<0, 1>() /*  keep object alive while iterator exists */)
   ;

src/core/matrices/codac2_MatrixBase.h

@@ -298,7 +298,7 @@ namespace codac2
 
       iterator begin()
       {
-        return const_cast<iterator>(const_cast<const MatrixBase<S,T,Rows,Cols>*>(this)->begin());
+        return _e.begin();
       }
 
       const_iterator begin() const
@@ -313,7 +313,7 @@ namespace codac2
 
       iterator end()
       {
-        return const_cast<iterator>(const_cast<const MatrixBase<S,T,Rows,Cols>*>(this)->end());
+        return _e.end();
       }
 
       EigenType _e;

tests/core/matrices/codac2_tests_Vector.cpp

@@ -16,5 +16,9 @@ using namespace codac2;
 
 TEST_CASE("Vector")
 {
-  
+  size_t i = 0;
+  Vector a(3), b {{-1},{2},{-3}};
+  for(const auto& bi : b)
+    a[i++] = bi;
+  CHECK(a == b);
 }