Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

BehaviourTreeInternalNode memory leak. #4

Open
GoogleCodeExporter opened this issue Mar 2, 2015 · 1 comment
Open

BehaviourTreeInternalNode memory leak. #4

GoogleCodeExporter opened this issue Mar 2, 2015 · 1 comment

Comments

@GoogleCodeExporter
Copy link

The way the examples uses BehaviourTreeInternalNode leads to memory leaks. the 
tree doesn't get deleted and it isn't possible to delete internalnode because 
of destructor being protected.
To delete it properly it has to be casted to the initializing pointer instance.
brain = new ParallelNode();

will need
if(ParallelNode temp = dynamic_cast<ParallelNode,brain)){
   delete temp;
}


Original issue reported on code.google.com by mart2...@gmail.com on 25 Jul 2013 at 12:12

@GoogleCodeExporter
Copy link
Author

of course the better solution is adding virtual destructor() to 
BehaviourTreeNode. Which should be done to any pure polymorphic classes.

Original comment by mart2...@gmail.com on 25 Jul 2013 at 4:33

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant