Use of R internal entry points in packages IBDsim and timeSeries

[Uwe Ligges]

On 10.08.2012 14:03, Magnus Dehli Vigeland wrote:
> On Wed, 01 Aug 2012 13:52:32 +0100, Prof Brian Ripley
> <ripley at stats.ox.ac.uk> wrote:
>> We see in R CMD check in current R-devel
>>
>> * checking foreign function calls ... WARNING
>> Foreign function call(s) with ‘PACKAGE’ argument in a base package:
>>   .C("find_interv_vec", ..., PACKAGE = "base")
>>
>> This is not part of the R API and will be removed shortly.
>>
>> Your uses of it are in any case incorrect and dangerous (e.g.
>> length() does not need to return an integer and might be NA: see its
>> help page).
>>
>> Please remove this from your code (and be aware that such calls are
>> seen as prime facie evidence of copying from the R sources, which you
>> have not acknowledged).
>
>
> Hi,
> I have now uploaded a new version of IBDsim (IBDsim_0.9-2.tar.gz) to
> CRAN without any such internal code calls.
>
> In normal use of this package findInternal() is called many hundred
> thousand times, so the cost reduction by calling the internal C code
> directly was quite substantial. I guess the best option is to create my
> own version in C, then.

Thanks, can you confirm you read and agree to the CRAN policies, please? 
Your package willbe moved to CRAN afterwards.

Best,
Uwe Ligges




>
> Thanks for your patience,
> Magnus
>
>