Horde3D

removeNode always returns false, whereas the documentation says it's supposed to return true in case of success, otherwise false.

I just changed the second return statement to return true

I only noticed this recently when I added assert statements to my code to make sure that all my Horde calls were succeeding

Thank you! This will be fixed immediately

One more thing

In SceneManager::removeNode, when the parent pointer is used it is check to make sure its not null:
but then later in the same function the pointer is used without checking:
should that line actually be like this?:



----------------
Also, when I reported the "return" bug, I was using 0.8. I've just upgraded my engine to use 0.9, but now it's crashing in SceneManager::removeNode on the 'delete' statement on this line:

The stack trace says: 0x10096865 std::allocator<SceneNode*>::~allocator (C:/Program Files/CodeBlocks/bin/../lib/gcc/mingw32/3.4.4/../../../../include/c++/3.4.4/bits/allocator.h:103)

I was going to report this as a bug, but if I compile the "chicago" sample, this crash does not happen - it only happens in my code, so I guess I've made a mistake somewhere! I'll let you know when I find out more (I'm guessing I'm calling removeNode twice on the same ID or something like that...).

This bug doesn't happen in the Chicago sample because it never calls removeNode!

If I add a member variable named _env to Application, and change this line:

and then add this line to Application::release:

then it crashes on the call to removeNode
again, the stack trace says std::allocator<SceneNode*>::~allocator and in the Horde code it crashes on the line:
// Delete node
delete _nodes[id - 1]; _nodes[id - 1] = 0x0;

Ok thanks. I will look into this issue and tell you when I find the problem. But this can still take some time since I am currently busy with restructuring the renderer.

I found the problem. The node to be deleted is not removed from the children list of its parent. Here is a quick fix for the problem. Just insert it between "Remove children" and "Mark dirty".

thank you very much!