Dirk Eddelbuettel
Dirk Eddelbuettel
> or is it manual? Might be in this package as I am conservative esp in old and working packages. In some other newer packages I do rely on roxygen2...
Turns out I may have been sloppy and or lazy in RApiSerialize and it can only be included from C++ code :grinning: so I added src/serializing.cpp and we now have...
Made a minor cleanup. Now ```sh $ Rscript -e 'library(digest); jenny
I think so too as it is a branch with a PR into my repo so AFAIUI github lets me. [ Commits, pushes. ] And it does. I am starting...
I think I would prefer to leave the digest R and C functions in the digest R package alone. They. Work. But we can and should re-use the infrastructure they...
We are having side-effects. I filed a (in hindsight: non-) bug report as `digest::digest` misbehaved: https://github.com/HenrikBengtsson/future/issues/740 Call that another vote for leaving `digest` alone and trying the 'direct' mods in...
> I think that makes this PR a no-go I was a bit surprised to see breakage. Which went away in the main branch. I haven't time or energy to...
That is a standard usage pattern of C++ code calling into C code. Given that R itself is only C (and Fortran) it is what we do all the time....
PS Looks like it is still pretty much the same code in R itself - lines 2811 to 2835 here (and GH is wonky and won't let me create a...
Similar to #220 I am going to close this. Happy to revisit as stated there.