<div dir="ltr"><div>Thank for the responses. <br></div><br>Subodh<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Sat, Aug 24, 2013 at 1:32 PM, Dirk Eddelbuettel <span dir="ltr"><<a href="mailto:edd@debian.org" target="_blank">edd@debian.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im"><br>
On 24 August 2013 at 09:04, <a href="mailto:romain@r-enthusiasts.com">romain@r-enthusiasts.com</a> wrote:<br>
| This is now fixed in svn.<br>
| The file did not have include guards and it was defining functions that<br>
| were too long to be inlined.<br>
<br>
</div>Thanks!<br>
<div class="im"><br>
| Since I don't want the user to have to link against an RcppArmadillo<br>
| library (like we do in Rcpp), I made these functions (SampleReplace, etc<br>
| ...) templates.<br>
| This should allow people to include this file more than once.<br>
|<br>
| The alternative would be to only have declarations of the<br>
| SampleReplace, etc ... functions in sample.h and definitions in a .cpp<br>
| file in RcppArmadillo, but then we would need users to mess with their<br>
| PKG_LIBS, and we would need to store a library we can link against. This<br>
| is already a lot of trouble to do this with Rcpp, so I'm not doing that<br>
| for RcppArmadillo.<br>
<br>
</div>Nobody suggested that. RcppArmadillo is, and should remain for the forseeable<br>
future, a templates-based headers-only library.<br>
<div class="im"><br>
| The other alternative is to not host sample in RcppArmadillo but in<br>
| another package. I don't have strong opinion on this, but sample is<br>
| somewhat of an outlier in RcppArmadillo. RcppArmadillo is just supposed<br>
| to make armadillo available. sample is an example use case.<br>
<br>
</div>Which is why it is off to the side and not included by default. I think that<br>
is a reasonably setup.<br>
<br>
A package of its own is possibly, but a tad on the heavy side for a single<br>
header file of a few dozen lines.  Other suggestions are surely welcome.<br>
<div class="im HOEnZb"><br>
Dirk<br>
<br>
--<br>
Dirk Eddelbuettel | <a href="mailto:edd@debian.org">edd@debian.org</a> | <a href="http://dirk.eddelbuettel.com" target="_blank">http://dirk.eddelbuettel.com</a><br>
</div><div class="HOEnZb"><div class="h5">_______________________________________________<br>
Rcpp-devel mailing list<br>
<a href="mailto:Rcpp-devel@lists.r-forge.r-project.org">Rcpp-devel@lists.r-forge.r-project.org</a><br>
<a href="https://lists.r-forge.r-project.org/cgi-bin/mailman/listinfo/rcpp-devel" target="_blank">https://lists.r-forge.r-project.org/cgi-bin/mailman/listinfo/rcpp-devel</a><br>
</div></div></blockquote></div><br><br clear="all"><br>-- <br>Acharya, Subodh<br>
</div>