eddelbuettel opened a new pull request, #842:
URL: https://github.com/apache/arrow-nanoarrow/pull/842
It looks like CRAN is unhappy about a remaining use of `ATTRIB`. Having
dealt with that myself, I think that `R_hasAttrib()`, added in R 4.6.0, should
be a fine replacement for you. I am not 100% sure it is ALTREP-safe --- it
says neither yay nor nay --- but I think it is as it does pretty much you do
here:
```
bool R_hasAttrib(SEXP x, SEXP name)
{
if (isScalarString(name)) name = installTrChar(STRING_ELT(name, 0));
if (! isSymbol(name))
error(_("'name' is not a symbol or a scalar string"));
if (name == R_NamesSymbol && isListWithNames(x))
return true;
SEXP attr = ATTRIB(x);
while (attr != R_NilValue) {
if (TAG(attr) == name)
return true;
attr = CDR(attr);
}
return false;
}
```
(The PR also adds an `importFrom` as R(-devel) CMD check complained.)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
