Skip to content

"needs export/exportS3Method" warning for conditionally-registered method #1880

@MichaelChirico

Description

@MichaelChirico

In {bit64}, we register some S3 methods conditionally, because the generic is not available on all supported R versions. This generates some noise in roxygenize():

✖ ops64.R:159: S3 method `chooseOpsMethod.integer64` needs @export or @exportS3Method tag.
✖ integer64.R:786: S3 method `print.bitstring` needs @export or @exportS3Method tag.
✖ patch64.R:103: S3 method `mtfrm.integer64` needs @export or @exportS3Method tag.

We do have exportS3Method() in the NAMESPACE, but it's in @rawNamespace because of the conditionality:

#' @rawNamespace if (getRversion() >= "4.3.0") S3method(chooseOpsMethod,integer64)
chooseOpsMethod.integer64 = function(x, y, mx, my, cl, reverse) {
  TRUE
}

What's the recommendation here?

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions