Hi Utkarsh

 If the package is an experiment and exploration, great - good for you 
 getting into this level of detail.
 However, if your goal is to get the dynamic prompt, a simpler mechanism
 that is almost equivalent is the R code

 invisible(addTaskCallback(function(...) { 
                             options(prompt = format(Sys.time(), "[%H:%M:%S] 
>"))
                             TRUE
                           }))

  The only difference is that when you hit return at the prompt with no 
expression,
  the prompt doesn't change (since there was no task).


  As for incorrectly using the API - yes. Since R CMD check is telling you this 
is a
  problem, it is officially a problem. This is of no consequence if the package 
works for you and any
  users of it, and the package is not to be hosted on CRAN. However, using this 
symbol in your code may not 
  work on all platforms, and is not guaranteed to work in the future (but 
probably will!). 

   D.
   

On 10/12/14, 8:13 AM, Utkarsh Upadhyay wrote:
> Hi,
> 
> I am trying to create a small extension for R here for embedding the
> current time on the R prompt: https://github.com/musically-ut/extPrompt
> 
> Things seem to be working overall, but `R CMD check .` raised a warning:
> 
>> File '[truncated]..Rcheck/extPrompt/libs/extPrompt.so’:
>>  Found non-API call to R: ‘ptr_R_ReadConsole’
>>
>> Compiled code should not call non-API entry points in R.
> 
> The concerned file is this:
> https://github.com/musically-ut/extPrompt/blob/master/src/extPrompt.c and
> occurs on line 38, I think [1].
> 
> I am trying to make the R_ReadConsole API call. However, since a different
> plugin (like mine) could have overridden it already, I do not want to
> directly invoke `R_ReadConsole` but the function which previously was at
> `ptr_R_ReadConsole`.
> 
> Is this an incorrect use of the API?
> 
> Thanks!
> 
> Best Regards,
> Utkarsh Upadhyay
> http://musicallyut.in
> 
> [1]: I am impressed at how well the static code analysis works for R CMD
> check.
> 
>       [[alternative HTML version deleted]]
> 
> ______________________________________________
> R-devel@r-project.org mailing list
> https://stat.ethz.ch/mailman/listinfo/r-devel
>

______________________________________________
R-devel@r-project.org mailing list
https://stat.ethz.ch/mailman/listinfo/r-devel

Reply via email to