On Sun, Nov 18, 2012 at 2:17 PM, Romain Francois
<[email protected]>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
>> [email protected]**project.org<[email protected]>
>> 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
> [email protected]**project.org<[email protected]>
> 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
_______________________________________________
Rcpp-devel mailing list
[email protected]
https://lists.r-forge.r-project.org/cgi-bin/mailman/listinfo/rcpp-devel
