Merge pull request #61 from Simple-Robotics/wjallet/topic/fix-eigenpy-3.3

jorisv · web-flow · commit 701c42c7c66a · 2024-01-23T16:35:26.000+01:00
Fix for eigenpy 3.3
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -8,6 +8,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ## [Unreleased]
 
 * Add missing dependencies in `package.xml`: pinocchio, eigen.
+* Fix `ConstraintObjectTpl::operator==` constness (mandatory for eigenpy 3.3)
 
 ## [0.3.4] - 2024-01-19
 
diff --git a/include/proxsuite-nlp/constraint-base.hpp b/include/proxsuite-nlp/constraint-base.hpp
@@ -140,7 +140,7 @@ template <typename _Scalar> struct ConstraintObjectTpl {
                       shared_ptr<ConstraintSet> set)
       : func_(func), set_(set) {}
 
-  bool operator==(const ConstraintObjectTpl &other) {
+  bool operator==(const ConstraintObjectTpl &other) const {
     return (func_ == other.func_) && (set_ && other.set_);
   }
 };

Original file line number	Diff line number	Diff line change
`@@ -140,7 +140,7 @@ template <typename _Scalar> struct ConstraintObjectTpl {`
`140`	`140`	`shared_ptr<ConstraintSet> set)`
`141`	`141`	`: func_(func), set_(set) {}`
`142`	`142`
`143`		`- bool operator==(const ConstraintObjectTpl &other) {`
	`143`	`+ bool operator==(const ConstraintObjectTpl &other) const {`
`144`	`144`	`return (func_ == other.func_) && (set_ && other.set_);`
`145`	`145`	`}`
`146`	`146`	`};`