<p>In <a href="https://github.com/mlpack/mlpack/pull/721#discussion_r70443843">src/mlpack/core/tree/rectangle_tree/x_tree_split_impl.hpp</a>:</p>
<pre style='color:#555'>> }
> }
>
> - std::sort(sorted.begin(), sorted.end(), structComp<ElemType>);
> + std::sort(sorted.begin(), sorted.end(),
> + [] (const std::pair<ElemType, size_t>& p1,
> + const std::pair<ElemType, size_t>& p2)
> + {
> + return p1.first < p2.first;
> + });
</pre>
<p>I replaced lambdas by a comparator. I define this function in <code>RStarTreeSplit</code> and <code>XTreeSplit</code>. These classes have a lot of duplicated code, I think we should not create a separate file for the comparator.</p>
<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/mlpack/mlpack/pull/721/files/ee2a8a1461a33edd830747dccde117ac2cfb12b7#r70443843">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe/AJ4bFNuljKIGUzhQWBUAcqjQsO_d9XOxks5qU55UgaJpZM4JKXF9">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AJ4bFGERAWCKUXjKNDJyo28LujD6x9Ngks5qU55UgaJpZM4JKXF9.gif" width="1" /></p>
<div itemscope itemtype="http://schema.org/EmailMessage">
<div itemprop="action" itemscope itemtype="http://schema.org/ViewAction">
<link itemprop="url" href="https://github.com/mlpack/mlpack/pull/721/files/ee2a8a1461a33edd830747dccde117ac2cfb12b7#r70443843"></link>
<meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>