[Rcpp-devel] Rcpp RMpfr mpreal to mpfr wrapping

[Dirk Eddelbuettel]

On 18 November 2012 at 21:17, Romain Francois wrote:
| 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) );

+1 -- had the same gut reaction.

Dirk

-- 
Dirk Eddelbuettel | edd at debian.org | http://dirk.eddelbuettel.com