Fix children[] array not deleted in deleteNodeRecurs function.

[xinyiz]

Fix issue where deleteNode children == __null assertion fails due to children[] array not deleted in deleteNodeRecurs function (https://github.com/OctoMap/octomap/issues/283, https://github.com/OctoMap/octomap/issues/287)

[spurnvoj]

Hi, this seems straightforward :) Is something missing to do merge?

[ahornung]

Yes, sorry for the delay. I wanted to check and add a few unit test beforehand, but that should happen soon.

[Levi-Armstrong]

We are also running into this assert. Friendly ping :)

[Levi-Armstrong]

@ahornung, Friendly ping :)

[christofschroeter]

Is there a reason not to do this cleanup in deleteNodeChild() already?
It semantically belongs to the deletion of the (last) child, and doing it outside may lead to having to add the same routine to other contexts from where deleteNodeChild is called.

[ahornung]

Good point! The only reason I can think of is the (probably) less efficient call of "pruneNode(...)", which does the cleanup once after deleting all 8 child nodes (instead of checking for every single call of deleteNodeChild if it's empty already).

[christofschroeter]

True, when you know you are deleting multiple children, you may not want to check after each deletion (which means iterating the 8 child pointers).

So I would suggest adding a parameter to deleteNodeChild() whether to cleanup, which in my opinion should have a default of true.
And if the cleanup is done internally, it might be an idea to return the result of the internal nodeHasChildren() from deleteNodeChild(), saving an extra query as in deleteNodeRecurs()

[vikgus1]

Friendly bump =)

[jasonir129]

bump! same issue :)