flexible-collision-library/fcl — pull request #19 by florent-lamiraux (2013-12-20 08:25:17+00:00)
Title: Add an accessor to shared pointer to CollisionGeometry member.

- classes CollisionObject and ContinuousCollisionObject,
- declare former accessors as deprecated.

I think that returning a pointer to a member of type shared pointer is not a good idea. If the user initializes his own shared pointer with the result of CollisionObject::getCollisionGeometry(), the object will be deleted twice. I quickly checked with the following code using valgrind.
```
#include

class A {
public:
A (int a) : a_ (a) {}
int a_;
};

int main ()
{
A* a = new A(1);
boost::shared_ptr a0 (a);
boost::shared_ptr a1 (a0.get ());
}
```