Merge pull request #530 from jorisv/topic/fix_eigen_serialization_duplicat

Avoid double fwd declaration in Pinocchio eigen.hpp

CHANGELOG.md

@@ -9,6 +9,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/).
 ### Fixed
 
 - CMake: fix assimp finder
+- Don't define GCC7 Boost serialization hack when `HPP_FCL_SKIP_EIGEN_BOOST_SERIALIZATION` is defined ([#530](https://github.com/humanoid-path-planner/hpp-fcl/pull/530))
 
 ## [2.4.1] - 2024-01-23
 

include/hpp/fcl/serialization/eigen.h

@@ -11,6 +11,8 @@
 #ifndef HPP_FCL_SERIALIZATION_EIGEN_H
 #define HPP_FCL_SERIALIZATION_EIGEN_H
 
+#ifndef HPP_FCL_SKIP_EIGEN_BOOST_SERIALIZATION
+
 #include <Eigen/Dense>
 
 #include <boost/serialization/split_free.hpp>
@@ -40,8 +42,6 @@ struct traits<boost::serialization::U> {
 namespace boost {
 namespace serialization {
 
-#ifndef HPP_FCL_SKIP_EIGEN_BOOST_SERIALIZATION
-
 template <class Archive, typename Scalar, int Rows, int Cols, int Options,
           int MaxRows, int MaxCols>
 void save(Archive& ar,
@@ -108,8 +108,9 @@ void serialize(Archive& ar,
   split_free(ar, m, version);
 }
 
-#endif  // ifned HPP_FCL_SKIP_EIGEN_BOOST_SERIALIZATION
 }  // namespace serialization
 }  // namespace boost
+//
+#endif  // ifned HPP_FCL_SKIP_EIGEN_BOOST_SERIALIZATION
 
 #endif  // ifndef HPP_FCL_SERIALIZATION_EIGEN_H