[mlpack-git] [mlpack/mlpack] Hilbert R tree (#664)

[lozhnikov]

> +    {
> +      localDataset = NULL;
> +      numValues = 0;
> +    }
> +  }
> +}
> +
> +template<typename TreeElemType>
> +template<typename TreeType>
> +void DiscreteHilbertValue<TreeElemType>::Copy(TreeType* dst, TreeType* src)
> +{
> +  DiscreteHilbertValue<TreeElemType> &dstVal = dst->AuxiliaryInfo().HilbertValue();
> +  DiscreteHilbertValue<TreeElemType> &srcVal = src->AuxiliaryInfo().HilbertValue();
> +
> +  dst.LocalDataset() = src.LocalDataset();
> +  dst.NumValues() = src.NumValues();

I agree, it is better to use a copy constructor and an operator =.

---
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/mlpack/mlpack/pull/664/files/f9127cea62d5ce1ad8d5f59931de108a2f7cdd9b#r67961063
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mailman.cc.gatech.edu/pipermail/mlpack-git/attachments/20160621/a4a93cfc/attachment.html>