linalg/ldlt : mark p() and pt() permutation matrix getters const (#180)

ManifoldFR · web-flow · commit eeb4888c430d · 2023-02-20T21:31:13.000+01:00
* linalg/ldlt : mark p() and pt() permutation matrix getters const

* linalg/ldlt : fix d_mut() returning DViewMut
diff --git a/include/proxsuite/linalg/dense/ldlt.hpp b/include/proxsuite/linalg/dense/ldlt.hpp
@@ -678,7 +678,7 @@ struct Ldlt
       Eigen::InnerStride<DYN>{ stride + 1 },
     };
   }
-  auto d_mut() noexcept -> DView
+  auto d_mut() noexcept -> DViewMut
   {
     return {
       ld_storage.ptr_mut(),
@@ -687,8 +687,8 @@ struct Ldlt
       Eigen::InnerStride<DYN>{ stride + 1 },
     };
   }
-  auto p() -> Perm { return { VecMapISize(perm.ptr(), dim()) }; }
-  auto pt() -> Perm { return { VecMapISize(perm_inv.ptr(), dim()) }; }
+  auto p() const -> Perm { return { VecMapISize(perm.ptr(), dim()) }; }
+  auto pt() const -> Perm { return { VecMapISize(perm_inv.ptr(), dim()) }; }
 
   /*!
    * Returns the memory storage requirements for a factorization of a matrix

Original file line number	Diff line number	Diff line change
`@@ -678,7 +678,7 @@ struct Ldlt`
`678`	`678`	`Eigen::InnerStride<DYN>{ stride + 1 },`
`679`	`679`	`};`
`680`	`680`	`}`
`681`		`- auto d_mut() noexcept -> DView`
	`681`	`+ auto d_mut() noexcept -> DViewMut`
`682`	`682`	`{`
`683`	`683`	`return {`
`684`	`684`	`ld_storage.ptr_mut(),`
`@@ -687,8 +687,8 @@ struct Ldlt`
`687`	`687`	`Eigen::InnerStride<DYN>{ stride + 1 },`
`688`	`688`	`};`
`689`	`689`	`}`
`690`		`- auto p() -> Perm { return { VecMapISize(perm.ptr(), dim()) }; }`
`691`		`- auto pt() -> Perm { return { VecMapISize(perm_inv.ptr(), dim()) }; }`
	`690`	`+ auto p() const -> Perm { return { VecMapISize(perm.ptr(), dim()) }; }`
	`691`	`+ auto pt() const -> Perm { return { VecMapISize(perm_inv.ptr(), dim()) }; }`
`692`	`692`
`693`	`693`	`/*!`
`694`	`694`	`* Returns the memory storage requirements for a factorization of a matrix`