[Rcpp-devel] FW: Rcpp attributes.cpp issue

[Latchezar (Lucho) Dimitrov]

What is the way to "send a proper patch for" you all "to review" Sorry for asking but it is not obvious on the list

"
- Why do you always answer questions with a question?
- Why not?
"

I apologize if I took too much of your all time


Thanks,
Latchezar

-----Original Message-----
From: Dirk Eddelbuettel [mailto:edd at debian.org] 
Sent: Sunday, April 19, 2015 9:06 AM
To: Latchezar (Lucho) Dimitrov
Cc: Kevin Ushey; rcpp-devel at lists.r-forge.r-project.org
Subject: Re: [Rcpp-devel] FW: Rcpp attributes.cpp issue


On 19 April 2015 at 04:54, Latchezar (Lucho) Dimitrov wrote:
| Yes, your understanding was correct J. My problem is I cannot w/o 
| going deep into the logic of the implementation to get the intent and 
| from it the type to cast. I am afraid I have to ask you: Are you sure 
| size_t is the type to cast in the context of attributes.cpp? Otherwise 
| by intuition I replaced 0U with 0UL (equivalent I believe, to size_t)  and it compiled.

If it builds for you, please send a proper patch (or pull request) for us to review.

 

| In the lack of understanding are there any ways to check the 
| correctness of so installed package Rcpp?

Do you know about R CMD check, and the over 900 tests it triggers?

Dirk

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