Skip to content

Commit

Permalink
compiler: remove lasts warnings
Browse files Browse the repository at this point in the history
  • Loading branch information
jorisv committed Jan 29, 2024
1 parent 4583a84 commit bea7367
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 1 deletion.
3 changes: 3 additions & 0 deletions bindings/python/parsers/urdf/geometry.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,10 @@ namespace pinocchio
MeshLoaderPtr mesh_loader = MeshLoaderPtr();
if (!py_mesh_loader.is_none()) {
#ifdef PINOCCHIO_WITH_HPP_FCL
PINOCCHIO_COMPILER_DIAGNOSTIC_PUSH
PINOCCHIO_COMPILER_DIAGNOSTIC_IGNORED_MAYBE_UNINITIALIZED
mesh_loader = bp::extract<::hpp::fcl::MeshLoaderPtr>(py_mesh_loader);
PINOCCHIO_COMPILER_DIAGNOSTIC_POP
#else
PyErr_WarnEx(PyExc_UserWarning, "Mesh loader is ignored because Pinocchio is not built with hpp-fcl", 1);
#endif
Expand Down
5 changes: 4 additions & 1 deletion unittest/spatial.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -713,7 +713,6 @@ PINOCCHIO_COMPILER_DIAGNOSTIC_IGNORED_MAYBE_UNINITIALIZED

// forceSet::se3Action
forceSet::se3Action(jMi,iF,jF);
PINOCCHIO_COMPILER_DIAGNOSTIC_POP
for( int k=0;k<N;++k )
BOOST_CHECK(jMi.act(Force(iF.col(k))).toVector().isApprox(jF.col(k)));

Expand All @@ -722,6 +721,7 @@ PINOCCHIO_COMPILER_DIAGNOSTIC_POP

forceSet::se3ActionInverse(jMi.inverse(),iF,jFinv);
BOOST_CHECK(jFinv.isApprox(jF));
PINOCCHIO_COMPILER_DIAGNOSTIC_POP

Matrix6N iF2 = Matrix6N::Random();
jF_ref += jMi.toDualActionMatrix() * iF2;
Expand Down Expand Up @@ -765,10 +765,13 @@ PINOCCHIO_COMPILER_DIAGNOSTIC_POP
BOOST_CHECK(jF.isApprox(jF_ref));

// Motion SET
PINOCCHIO_COMPILER_DIAGNOSTIC_PUSH
PINOCCHIO_COMPILER_DIAGNOSTIC_IGNORED_MAYBE_UNINITIALIZED
Matrix6N iV = Matrix6N::Random(),jV,jV_ref,jVinv,jVinv_ref;

// motionSet::se3Action
motionSet::se3Action(jMi,iV,jV);
PINOCCHIO_COMPILER_DIAGNOSTIC_POP
for( int k=0;k<N;++k )
BOOST_CHECK(jMi.act(Motion(iV.col(k))).toVector().isApprox(jV.col(k)));

Expand Down

0 comments on commit bea7367

Please sign in to comment.