Add smart ptr typedefs

[bmagyar]

This adds the common typedefs that already exist for the rest of the classes in the urdf namespace.

[jspricke]

I would propose to use std::shared_ptr here, as urdf::Model inherits from urdf::ModelInterface and urdfdom_headers switched to std::shared_ptr already.

[bmagyar]

Hiya,
I am fully aware of that. Moving to these macros consistently in
ros_control is what I've been working on recently in order to ease
"unboosting" when it happens. The stuff in the urdf headers haven't been
released yet, this brings the local urdf class up to date w the released
version of that.

On 11 Sep 2016 11:53, "Jochen Sprickerhof" notifications@github.com wrote:

I would propose to use std::shared_ptr here, as urdf::Model inherits from
urdf::ModelInterface and urdfdom_headers switched to std::shared_ptr
already.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#153 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/ADXH4c7Yz0lmq3ziGOVOZgRRPe4vjaAsks5qo92agaJpZM4J0aMq
.

[de-vri-es]

This PR would be very welcome. MoveIt is currently being migrated to std smart pointers, so any remaining explicit boost::shared_ptr is an eyesore.

More importantly, once urdf switches to std::shared_ptr, explicit usage of boost::shared_ptr will stop compiling. It would be nice to be able to prepare for that by using these typedefs before the actual switch is done.

[bmagyar]

@wjwwood ?

[davetcoleman]

When will these pointers be switched to std? Currently in Lunar all of the shared_ptrs in urdfdom/urdfdom_headers uses std, but this one Model class still uses boost.