deleting bodys, feature request

i'm using 0.4 RC3 and when i used this function:

//////////////////////////////////////////////////////////////////////

void scene::destroyBody(Ogre::String _name) {

mBodies.destroyBody(mBodies.find(_name));

/*std::map<std::string, body*>::iterator it = mBodies.find(_name);

if (it == mBodies.end()) {
error::getSingleton().Throw("Attempting to delete a body '" + _name + "' that does not exist!",error::WARNING, _name, "scene::destroyBody");
return;
}

delete it->second;
mBodies.erase(it);*/

}


there is no way to know if your attempt to delete the body was successful....
would be nice to return bool
and since mBodies.destroyBody(mBodies.find(_name)); returns true on success, the function could end like this:


//////////////////////////////////////////////////////////////////////

bool scene::destroyBody(Ogre::String _name) {

return mBodies.destroyBody(mBodies.find(_name));
}



Another comment: i've seen that when you create two bodys with the same name, the first one doesnt get its coordinates updated anymore(i'm talking about its Ogre's scenenode).
Seems that the first one just gets "unreferenced" in the list of bodys to be updated their visual representation. But the physics body is still alive and beeing correctly simulated (i can see them with the debug wireframe).

In my opinion it is a bit bug-prone, shouldnt be better to throw an exception telling you the name of the body duplicated when you register two bodies with the same name?

Thanks for your great work betajaen!

Good points with both of them.

Second one is just a bug really, the newer body replaces the previous body position in the map. The old body is physically there, (the NxActor isn't deleted) it's just not known anymore, and since it's not known. It doesn't get deleted on shutdown so we get a leak or two.