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

Fix memory leak when dual tree search and treeOwner == true #731

Merged
merged 1 commit into from Jul 20, 2016

Conversation

Projects
None yet
2 participants
@MarcosPividori
Contributor

MarcosPividori commented Jul 20, 2016

No description provided.

@rcurtin rcurtin merged commit 54c48db into mlpack:master Jul 20, 2016

1 of 2 checks passed

continuous-integration/travis-ci/pr The Travis CI build failed
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details

rcurtin added a commit that referenced this pull request Jul 20, 2016

@rcurtin

This comment has been minimized.

Show comment
Hide comment
@rcurtin

rcurtin Jul 20, 2016

Member

Nice catch. I also applied the fix to RangeSearch and RASearch in 9dc7de5.

Member

rcurtin commented Jul 20, 2016

Nice catch. I also applied the fix to RangeSearch and RASearch in 9dc7de5.

rcurtin added a commit that referenced this pull request Jul 20, 2016

@MarcosPividori MarcosPividori changed the title from Fix memory leaf when dual tree search and treeOwner == true to Fix memory leak when dual tree search and treeOwner == true Jul 22, 2016

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