[mlpack] [mlpack/mlpack] Added copy constructor, move constructor and copy assignment operator. (#826)

Ryan Curtin notifications at github.com
Mon Dec 12 11:46:42 EST 2016


Hi there,

Thanks for the contribution.  Could you please add some tests for these?  Take a look at some tests I recently wrote in a different branch: https://github.com/rcurtin/mlpack/blob/bindings/src/mlpack/tests/lsh_test.cpp#L840

Once there are working tests I'll give it a full review.

-- 
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/826#issuecomment-266482640
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://knife.lugatgt.org/pipermail/mlpack/attachments/20161212/8e2ceb27/attachment.html>


More information about the mlpack mailing list