New NOTE nag from R-devel

[eddelbuettel]

We now get:

Version: 1.1.1
Check: compiled code
Result: NOTE
File ‘Rcpp/libs/Rcpp.so’:
Found non-API call to R: ‘R_UnboundValue’

Compiled code should not call non-API entry points in R.

See ‘Writing portable packages’ in the ‘Writing R Extensions’ manual,
and section ‘Moving into C API compliance’ for issues with the use of
non-API entry points.
Flavor: r-devel-linux-x86_64-debian-gcc

In current svn sources (I am at r89737) I see no mention of an alternative. We use it in a few places, including as the last argument in R_getVarEx() which, I am fairly certain, was just added this winter as part API cleanups. Sigh, clearly cannot win this.

[Enchufa2]

The cases are:

Rcpp/inst/include/Rcpp/api/meat/Rcpp_eval.h

Line 59 in e5b03df

if (identity == R_UnboundValue) {

can be removed. Rf_findFun (above) never returns R_UnboundValue.

Rcpp/inst/include/Rcpp/Environment.h

Line 103 in e5b03df

SEXP res = R_getVarEx(nameSym, env, FALSE, R_UnboundValue);

There's no reason to use R_UnboundValue here. We can just use R_NilValue and return res directly in this path.

Rcpp/inst/include/Rcpp/Function.h

Line 77 in e5b03df

if (env == R_UnboundValue) {

Similarly, we can use R_NilValue here too.

Rcpp/inst/include/Rcpp/Promise.h

Line 52 in e5b03df

if( val == R_UnboundValue ) throw unevaluated_promise() ;

We need to move away from PRVALUE (see this, and when we do, R_UnboundValue will be no longer needed.

And that's all I think.