[Rcpp-devel] Rcpp RMpfr mpreal to mpfr wrapping

Thell Fowler tbfowler4 at gmail.com
Sun Nov 18 22:25:44 CET 2012


On Sun, Nov 18, 2012 at 2:17 PM, Romain Francois
<romain at r-enthusiasts.com>wrote:

> Hello,
>
> You don't need to wrap an "Rcpp::S4", as these are already R objects, so
> an implicit conversion to SEXP already exists.
>
> Same here:
>
>
> vS4.slot("exp") = wrap( exp );
>
> you don't need this wrap, you should be able to just use:
>
> vS4.slot("exp") = exp ;
>
> for the same reason, exp is already an R object.
>
> Same for:
>
> vS4.slot("d") = wrap( d );
>
> and:
>
>
> ans.slot(".Data") = List::create( wrap(vS4) );
>
>
> True, true.  Thanks for pointing that out.


>
> I see that you use "auto" from C++11. Cool, but this will make it hard to
> ship the package to CRAN.
>
> Romain
>
>
> Is there some kind of a stance against 'auto'?

 --->8 ---

>
>> Sincerely,
>> Thell
>>
>>
>> ______________________________**_________________
>> Rcpp-devel mailing list
>> Rcpp-devel at lists.r-forge.r-**project.org<Rcpp-devel at lists.r-forge.r-project.org>
>> https://lists.r-forge.r-**project.org/cgi-bin/mailman/**
>> listinfo/rcpp-devel<https://lists.r-forge.r-project.org/cgi-bin/mailman/listinfo/rcpp-devel>
>>
>>
>
> --
> Romain Francois
> Professional R Enthusiast
> +33(0) 6 28 91 30 30
>
> R Graph Gallery: http://gallery.r-enthusiasts.**com<http://gallery.r-enthusiasts.com>
> `- http://bit.ly/SweN1Z : SuperStorm Sandy
>
> blog:            http://romainfrancois.blog.**free.fr<http://romainfrancois.blog.free.fr>
> |- http://bit.ly/RE6sYH : OOP with Rcpp modules
> `- http://bit.ly/Thw7IK : Rcpp modules more flexible
>
>
> ______________________________**_________________
> Rcpp-devel mailing list
> Rcpp-devel at lists.r-forge.r-**project.org<Rcpp-devel at lists.r-forge.r-project.org>
> https://lists.r-forge.r-**project.org/cgi-bin/mailman/**
> listinfo/rcpp-devel<https://lists.r-forge.r-project.org/cgi-bin/mailman/listinfo/rcpp-devel>
>



-- 
Sincerely,
Thell
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.r-forge.r-project.org/pipermail/rcpp-devel/attachments/20121118/37f455d2/attachment.html>


More information about the Rcpp-devel mailing list