Merge pull request #429 from stack-of-tasks/namespace

do not use namespace eigenpy and define bp
stack-of-tasks · Jan 30, 2024 · ebde399 · ebde399
2 parents e55c568 + c60cf2a
commit ebde399
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -11,6 +11,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/).
 
 ### Fixed
 - Fix the issue of missing exposition of Eigen types with __int64 scalar type ([#426](https://github.com/stack-of-tasks/eigenpy/pull/426))
+- Fix namespace use in unittest/std_pair.cpp ([#429](https://github.com/stack-of-tasks/eigenpy/pull/429))
 
 ## [3.3.0] - 2024-01-23
 

diff --git a/unittest/std_pair.cpp b/unittest/std_pair.cpp
@@ -18,12 +18,10 @@ std::pair<T1, T2> copy(const std::pair<T1, T2>& pair) {
 }
 
 BOOST_PYTHON_MODULE(std_pair) {
-  using namespace eigenpy;
-
-  enableEigenPy();
+  eigenpy::enableEigenPy();
 
   typedef std::pair<int, double> PairType;
-  StdPairConverter<PairType>::registration();
+  eigenpy::StdPairConverter<PairType>::registration();
 
   bp::def("std_pair_to_tuple", std_pair_to_tuple<int, double>);
   bp::def("copy", copy<int, double>);