Prevent 'bad weak_ptr exception' when copying return from getRobotNameHelper()

The RobotNameHelper uses shared_from_this, which makes copying the class very dangerous. E.g. an 'auto rnh = getRobotNameHelper();' inevitably lead to this error as the the rnh was copied (returned const ref and auto drops const refs --> the type of rnh was RobotNameHelper). To prevent this happening to unknowing users, it now returns a shared_ptr:

Signed-off-by: robdekon_h2t robdekon@kit.edu

source/RobotAPI/libraries/GraspingUtility/box_to_grasp_candidates.cpp

@@ -302,8 +302,8 @@ namespace armarx
 
 namespace armarx
 {
-    box_to_grasp_candidates::box_to_grasp_candidates(const armarx::RobotNameHelper& rnh, const VirtualRobot::RobotPtr& robot)
-        : _rnh{std::make_shared<armarx::RobotNameHelper>(rnh)}, _robot{robot}
+    box_to_grasp_candidates::box_to_grasp_candidates(armarx::RobotNameHelperPtr rnh, const VirtualRobot::RobotPtr& robot)
+        : _rnh{std::move(rnh)}, _robot{robot}
     {
         ARMARX_CHECK_NOT_NULL(_robot);
     }