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 | [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

Reply via email to