The matrixStats::validateIndices() does not really have to be exported. It is currently only used in package tests to validate the correctness of how we handle indices at the C level. Because of this, it could equally well be an internal function.
The safest is probably to go through a proper deprecation process:
[x] Deprecate the function, i.e. have it produce a warning if called from other than package tests (=> matrixStats 0.61.0)
[ ] Later, make it defunct, i.e. produce an error if called elsewhere (=> matrixStats 0.63.0)
The
matrixStats::validateIndices()
does not really have to be exported. It is currently only used in package tests to validate the correctness of how we handle indices at the C level. Because of this, it could equally well be an internal function.The safest is probably to go through a proper deprecation process: