Re: [Rd] winbuilder warning message wrt function pointers

2017-12-29 Thread William Dunlap via R-devel
You can legally cast a function pointer to another function pointer, where the signatures differ. (It is not legal to cast between data and function pointers.) I would make typedefs for the various signatures, as the casting syntax is more readable then. Bill Dunlap TIBCO Software wdunlap tibco.c

Re: [Rd] winbuilder warning message wrt function pointers

2017-12-29 Thread Therneau, Terry M., Ph.D.
Bill,   That's a very nice solution.  It is both cleaner looking and preferable to track R's .h files. However, some of my routines don't have void * as the return type (two are int *), and Rdynload has    typedef void * (*DL_FUNC)(); Will this untruth mess anything up? Terry T. On 12/29/

Re: [Rd] winbuilder warning message wrt function pointers

2017-12-29 Thread William Dunlap via R-devel
And remove the cast on the return value of R_GETCCallable. And check that your function is found before using it. #include #include #include void bdsmatrix_prod4(int nrow,int nblock, int *bsize, double *bmat, double *rmat, int nfrail, double *y)

Re: [Rd] winbuilder warning message wrt function pointers

2017-12-29 Thread William Dunlap via R-devel
Try changing static void (*fun)() = NULL; to DL_FUNC fun = NULL; Bill Dunlap TIBCO Software wdunlap tibco.com On Fri, Dec 29, 2017 at 5:14 AM, Therneau, Terry M., Ph.D. < thern...@mayo.edu> wrote: > I've recently updated the coxme package, which calls internal routines > from the bdsmatrix p

Re: [Rd] Typos in ?regex

2017-12-29 Thread Uwe Ligges
Thanks, fixed in R-devel and R-patched. Best, Uwe Ligges On 22.12.2017 15:28, Korpela Mikko (MML) wrote: I found three little typos in the ?regex manual. Apologies for the lack of a diff, as the utility is not (yet) installed on this machine. 1. "There is a also" should probably be "There is

Re: [Rd] data compression in a package

2017-12-29 Thread Dirk Eddelbuettel
On 29 December 2017 at 07:59, Therneau, Terry M., Ph.D. wrote: | which is already taken care of)." The survival package has about 2M of .rda files and | 2.2M of vignettes so I'm still under the 5M boundary, but R CMD check nags about it. The nags are genuinely irritating, especially as they a

Re: [Rd] data compression in a package

2017-12-29 Thread Therneau, Terry M., Ph.D.
On 12/29/2017 07:34 AM, Dirk Eddelbuettel wrote: On 29 December 2017 at 07:23, Therneau, Terry M., Ph.D. wrote: | The submission guide has the following cryptic (to me) sentence: |    "Reasonable compression should be used for data (not just .rda files) " | | The survival pacakge has a fairly

Re: [Rd] Numerical stability in chisq.test

2017-12-29 Thread Jan Motl
Hi, there is also PR#8224, which seems to be relevant. I executed the following code: ## Modify the function chisq.test2 <- edit(chisq.test) # Modify to use increasing order of sorting at line 57 ## PR#8224 (patological contingency table) m <- matrix(c(1,0,7,16),2,2); # Original original <-

Re: [Rd] data compression in a package

2017-12-29 Thread Dirk Eddelbuettel
On 29 December 2017 at 07:23, Therneau, Terry M., Ph.D. wrote: | The submission guide has the following cryptic (to me) sentence: |    "Reasonable compression should be used for data (not just .rda files) " | | The survival pacakge has a fairly large number of data files --- exactly what should

[Rd] data compression in a package

2017-12-29 Thread Therneau, Terry M., Ph.D.
The submission guide has the following cryptic (to me) sentence:    "Reasonable compression should be used for data (not just .rda files) " The survival pacakge has a fairly large number of data files --- exactly what should I be doing?   xz compression? Terry T. _

[Rd] winbuilder warning message wrt function pointers

2017-12-29 Thread Therneau, Terry M., Ph.D.
I've recently updated the coxme package, which calls internal routines from the bdsmatrix package.  (It is in fact mentioned as an example of this in the Extensions manual.) The call connections are a blocks like this, one for each of the 9 called C routines. void bdsmatrix_prod4(int nrow,    i