<p>In <a href="https://github.com/mlpack/mlpack/pull/664#discussion_r65606357">src/mlpack/core/tree/rectangle_tree/rectangle_tree.hpp</a>:</p>
<pre style='color:#555'>&gt; @@ -91,8 +93,8 @@ class RectangleTree
&gt;    std::vector&lt;size_t&gt; points;
&gt;    //! The local dataset
&gt;    MatType* localDataset;
&gt; -  //! The class that performs the split of the node.
&gt; -  SplitType&lt;RectangleTree&gt; split;
&gt; +  //! A tree-specific information
&gt; +  AuxiliaryInformationType&lt;RectangleTree&gt; *auxiliaryInfo;
</pre>
<p>Are you sure we want to allocate this on the heap?  We then have to manage the memory and make sure it is valid.  I think usually the <code>AuxiliaryInformationType</code> classes will be small (looks like the one for the Hilbert R tree will be maybe 32 bytes), so I don't know if this is worth doing.  (A few more comments about the AuxiliaryInformationType are coming but I have to step out, so it may be a little while until I am able to write those comments.)</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">&mdash;<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/664/files/ab890dff6b4b617b5309af53950c6cac97d994d6#r65606357">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe/AJ4bFLxUyzH7-O2NRfCIraYJiYcOMjgEks5qHzPngaJpZM4IrlzT">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AJ4bFFFgnlPyn7d5Sv-KTwJdY7JC40FYks5qHzPngaJpZM4IrlzT.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/664/files/ab890dff6b4b617b5309af53950c6cac97d994d6#r65606357"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>