Direct call to S3 method from bit64

[MichaelChirico]

Here are some direct calls to an S3 method

nanotime/R/nanoduration.R

Line 650 in 5fe3414

as.nanoduration(c.integer64(...))

nanotime/R/nanotime.R

Line 657 in 5fe3414

nanotime(c.integer64(...))

This will stop working in the future when this symbol is un-exported. Is there a reason not to let S3 dispatch "just work" in this case?

See r-lib/bit64#76

[eddelbuettel]

Then things break as revealed by a two-minute investigation. Context matters. You should have shown the complete (symmetric) functions:

 c.nanoduration <- function(...) {
     as.nanoduration(c.integer64(...))
 }

I tried both c(as.integer64(...)) and c(...); neither approach made it far under R CMD check. All this is tricky business as we have to mix S4 and S3 semantics. We like the status quo, it worked for eight+ years.

[MichaelChirico]

Thanks Dirk. Rest assured I have no interest in breaking working code :)

The timeline here is long -- I won't even touch this in the next release, and will only look at adding warnings in the subsequent release before finally removing the export after that. I will of course retain c.integer64 exported as long as we don't have a fix for you.

I've filed r-lib/bit64#150 to track this specific issue.