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

2015-04-20 Thread Latchezar (Lucho) Dimitrov
rrier.o installing to /zpool.1/transient/temp/R/build-patched/library/Rcpp/libs ** R ** inst ** preparing package for lazy loading ** help *** installing help indices ** building package indices ** installing vignettes ** testing if installed package can be loaded * DONE (Rcpp) > -Original Me

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

2015-04-20 Thread Dirk Eddelbuettel
Thanks, I see now. Could you try std::string::size_type instead of size_t ? See eg http://stackoverflow.com/questions/1181079/stringsize-type-instead-of-int I would hate to see us drop generality here just because you happen to work with a peculiar compiler. Dirk -- http://dirk.eddelbuettel.

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

2015-04-20 Thread Latchezar (Lucho) Dimitrov
.r-forge.r-project.org Subject: RE: [Rcpp-devel] FW: Rcpp attributes.cpp issue http://jungels.net/articles/diff-patch-ten-minutes.html Google hit number one for 'how to create a patch file'. We really prefer diff -u. Your patch is __useless__ as it doesn't even say which fi

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

2015-04-20 Thread Dirk Eddelbuettel
http://jungels.net/articles/diff-patch-ten-minutes.html Google hit number one for 'how to create a patch file'. We really prefer diff -u. Your patch is __useless__ as it doesn't even say which file(s) you modified. Dirk -- http://dirk.eddelbuettel.com | @eddelbuettel | e...@debian.org

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

2015-04-20 Thread Latchezar (Lucho) Dimitrov
delbuettel [mailto:e...@debian.org] Sent: Monday, April 20, 2015 1:33 PM To: Latchezar (Lucho) Dimitrov Cc: Dirk Eddelbuettel; Kevin Ushey; rcpp-devel@lists.r-forge.r-project.org Subject: RE: [Rcpp-devel] FW: Rcpp attributes.cpp issue On 20 April 2015 at 17:08, Latchezar (Lucho) Dimitrov wrote: | What is

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

2015-04-20 Thread Dirk Eddelbuettel
On 20 April 2015 at 17:08, Latchezar (Lucho) Dimitrov wrote: | 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 Rcpp code is on GiHub as even a casual Google query would reveal. How to contribute via GitHub is explained in numer

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

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

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

2015-04-19 Thread Dirk Eddelbuettel
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 cas

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

2015-04-18 Thread Latchezar (Lucho) Dimitrov
: Latchezar (Lucho) Dimitrov Cc: rcpp-devel@lists.r-forge.r-project.org Subject: Re: [Rcpp-devel] FW: Rcpp attributes.cpp issue IIUC, from the build errors you're seeing, the compiler can't decide if it should convert `0U` (unsigned int) to an `unsigned long` or a `char*` (e.g. as a nu

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

2015-04-18 Thread Kevin Ushey
IIUC, from the build errors you're seeing, the compiler can't decide if it should convert `0U` (unsigned int) to an `unsigned long` or a `char*` (e.g. as a null pointer). In other words, these two overloads are competing: void insert (iterator p, size_t n, char c); string& insert (size_t pos,