<blockquote>
<p>Ok, great, I am happy to add this. I made a few changes after the merge. <a href="https://github.com/mlpack/mlpack/commit/77d098c4774107e1b71e214dcc56abf99c425abb" class="commit-link"><tt>77d098c</tt></a> removes the HAS_OPENMP CMake variable (but not the compiler definition), since it isn't used anywhere. Let me know if that causes any problems; maybe I misunderstood where you are using it. (Note that the line SET(HAS_OPENMP 1) sets the CMake variable, but add_definitions(-DHAS_OPENMP) makes HAS_OPENMP available as a macro in the code.)</p>
</blockquote>

<p>Cool, I didn't know this. I thought SET(HAS_OPENMP 1) was the equivalent to <code>#define HAS_OPENMP 1</code>. I'm not using the CMake variable - only the C++ macro so this should not be a problem.</p>

<blockquote>
<p><a href="https://github.com/mlpack/mlpack/commit/ca1e3ecba3412a14ff25b2ac16011dd8f2a3bfef" class="commit-link"><tt>ca1e3ec</tt></a> and <a href="https://github.com/mlpack/mlpack/commit/aaefadd7cfd1f0f7d4ad0118d7e23cea76383397" class="commit-link"><tt>aaefadd</tt></a> are minor style fixes, but the last one, <a href="https://github.com/mlpack/mlpack/commit/44c7c3578f5fddb1dc15b42e8c52861ba987a37e" class="commit-link"><tt>44c7c35</tt></a>, concerns me a bit; I changed the line avgIndicesReturned = avgIndicesReturned + refIndices.n_elem to avgIndicesReturned += refIndices.n_elem. It worked on my system, and it appears that it worked on AppVeyor, but maybe there is something I overlooked, so let me know if there is. I guess we will find out soon enough if I broke anything like this!</p>
</blockquote>

<p>I don't know why I did it like this. I just copied the OpenMP tutorial's code to be certain what I was doing was right - I feared reductions would depend on specific syntax. That's not true, so your fix is equivalent but easier to read.</p>

<p>Thanks for the merge :)</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/700#issuecomment-231671712">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe/AJ4bFPym-FeWG4ictjYR2EKHRrW2UAO7ks5qUf5xgaJpZM4I5KSz">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AJ4bFNd7Ru0DC_giOTJ4reKDbAuHlIxBks5qUf5xgaJpZM4I5KSz.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/700#issuecomment-231671712"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>