<blockquote>
<p>Hmm, do you think that we should modify prereqs.hpp or something to define ARMA_USE_CXX11 when using Visual Studio 2015?</p>
</blockquote>

<p>I think this is a good idea if everything still compile correctly</p>

<blockquote>
<p>Also maybe I am missing it but I don't see where std::move() is used, so maybe the comment is no longer necessary?</p>
</blockquote>

<p>I do not use std::move to move them explicitly because this may generate less effective binary, compiler will try to do RVO if they could, if they could not, they would move the variable back, compiler is smart enough to find out they are local variable only needed by that function. </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 or <a href="https://github.com/mlpack/mlpack/pull/523#issuecomment-209546497">view it on GitHub</a><img alt="" height="1" src="https://github.com/notifications/beacon/AJ4bFImMQ53f00ZsyoEreG5luln173pvks5p3SFugaJpZM4HhWF3.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/523#issuecomment-209546497"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>