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