Skip to content

Remove setrev function #3704

@MichaelChirico

Description

@MichaelChirico

As pointed out in a comment on #3688:

Trying to do coverage and came across this function: setrev

I don't see it used anywhere and it's not exported... shall we delete it? Otherwise we should file an issue to have it exported.

grep to show it's unused:

grep -r "setrev" . | grep -v "Rproj" | grep -v "tests/tests" | grep -v "Binary"
./R/setkey.R:setrev = function(x) .Call(Csetrev, x)
./src/init.c:SEXP setrev();
./src/init.c:{"Csetrev", (DL_FUNC) &setrev, -1},
./src/reorder.c:SEXP setrev(SEXP x) {

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions