[mlpack-git] master: I suppose we should exercise at least some caution in the destructor. (8d209f6)

gitdub at big.cc.gt.atl.ga.us gitdub at big.cc.gt.atl.ga.us
Thu Mar 5 22:01:22 EST 2015


Repository : https://github.com/mlpack/mlpack

On branch  : master
Link       : https://github.com/mlpack/mlpack/compare/904762495c039e345beba14c1142fd719b3bd50e...f94823c800ad6f7266995c700b1b630d5ffdcf40

>---------------------------------------------------------------

commit 8d209f60057fc4e907ff3dfa00fb1e5b28ae61ee
Author: Ryan Curtin <ryan at ratml.org>
Date:   Sun Oct 12 21:09:02 2014 +0000

    I suppose we should exercise at least some caution in the destructor.


>---------------------------------------------------------------

8d209f60057fc4e907ff3dfa00fb1e5b28ae61ee
 src/mlpack/methods/kmeans/pelleg_moore_kmeans_impl.hpp | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/mlpack/methods/kmeans/pelleg_moore_kmeans_impl.hpp b/src/mlpack/methods/kmeans/pelleg_moore_kmeans_impl.hpp
index e7fd385..4a2bf83 100644
--- a/src/mlpack/methods/kmeans/pelleg_moore_kmeans_impl.hpp
+++ b/src/mlpack/methods/kmeans/pelleg_moore_kmeans_impl.hpp
@@ -39,7 +39,8 @@ PellegMooreKMeans<MetricType, MatType>::PellegMooreKMeans(
 template<typename MetricType, typename MatType>
 PellegMooreKMeans<MetricType, MatType>::~PellegMooreKMeans()
 {
-  delete tree;
+  if (tree)
+    delete tree;
 }
 
 // Run a single iteration.



More information about the mlpack-git mailing list