[mlpack-git] [mlpack/mlpack] Use move semantics to set a given reference tree in NeighborSearch class. (#765)

Ryan Curtin notifications at github.com
Mon Aug 29 17:03:34 EDT 2016


Other than the comments I made, I think everything is good here and it's ready for merge.  We should at some point roll out the same changes to `RASearch`, `RangeSearch`, `FastMKS`, and `DualTreeBoruvka`, but we can open a ticket that says that and just merge these changes now---your call.

-- 
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/765#issuecomment-243255498
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mailman.cc.gatech.edu/pipermail/mlpack-git/attachments/20160829/956989c6/attachment.html>


More information about the mlpack-git mailing list