Skip to content
This repository has been archived by the owner on Nov 13, 2017. It is now read-only.

Commit

Permalink
Keep PR#702 to just the minimum needed to fix #592
Browse files Browse the repository at this point in the history
  • Loading branch information
Alain Sanguinetti committed Jul 4, 2016
1 parent 9283d9a commit 2da142f
Show file tree
Hide file tree
Showing 2 changed files with 14 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -62,6 +62,8 @@ class KinematicsPluginLoader
{
}

~KinematicsPluginLoader();

/** \brief Use a default kinematics solver (\e solver_plugin) for
all the groups in the robot model. The default timeout for the
solver is \e solve_timeout and the default number of IK attempts
Expand Down
12 changes: 12 additions & 0 deletions planning/kinematics_plugin_loader/src/kinematics_plugin_loader.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -76,6 +76,14 @@ class KinematicsPluginLoader::KinematicsLoaderImpl
}
}

/**
* \brief Explicit destructor to avoid bugs in unloading the library
*/
~KinematicsLoaderImpl( )
{
kinematics_loader_.reset(); // we delete it manually
}

/**
* \brief Helper function to decide which, and how many, tip frames a planning group has
* \param jmg - joint model group pointer
Expand Down Expand Up @@ -243,6 +251,10 @@ void kinematics_plugin_loader::KinematicsPluginLoader::status() const
ROS_INFO("Loader function was never required");
}

kinematics_plugin_loader::KinematicsPluginLoader::~KinematicsPluginLoader()
{
}

robot_model::SolverAllocatorFn kinematics_plugin_loader::KinematicsPluginLoader::getLoaderFunction()
{
moveit::tools::Profiler::ScopedStart prof_start;
Expand Down

0 comments on commit 2da142f

Please sign in to comment.