Re: [Rd] Good practice for packages with Fortran and C code

2024-10-25 Thread Ivan Krylov via R-devel
В Fri, 25 Oct 2024 15:03:54 -0500 f...@fharrell.com пишет: > Now I find that I can get rid of init.c, and change NAMESPACE to use > useDynLib(package name, list of compiled routine names) (without > .registration and .fixes) after making sure the Fortran and C > routines are not named the same as

Re: [Rd] [External] Re: Could .Primitive("[") stop forcing R_Visible = TRUE?

2024-10-25 Thread Ivan Krylov via R-devel
Dear Luke, Thank you very much for taking the time to write an exhaustive answer! They are always a pleasure to read on the R mailing lists. On Fri, 25 Oct 2024 11:50:34 -0500 (CDT) luke-tier...@uiowa.edu wrote: > So there is a discrepancy between interpreted and compiled code which > is a bug t

[Rd] Good practice for packages with Fortran and C code

2024-10-25 Thread fh
My older packages have some Fortran and C routines. The Fortran is ancient and I am started to convert it to Fortran 2018 (thanks ChatGPT). To have a mixture of Fortran and C code in a package I have had a src/init.c file that provides names for routines of the form F77_name, and I called th

Re: [Rd] [External] Re: Could .Primitive("[") stop forcing R_Visible = TRUE?

2024-10-25 Thread luke-tierney--- via R-devel
On Fri, 25 Oct 2024, Ivan Krylov via R-devel wrote: В Fri, 25 Oct 2024 08:39:39 -0400 Duncan Murdoch пишет: Surely you or they should be the ones to run the test across all of CRAN? That's fair. The question is, is there a fundamental reason I overlooked to deny such a change? Except for po

Re: [Rd] Could .Primitive("[") stop forcing R_Visible = TRUE?

2024-10-25 Thread Ivan Krylov via R-devel
В Fri, 25 Oct 2024 08:39:39 -0400 Duncan Murdoch пишет: > Surely you or they should be the ones to run the test across all of > CRAN? That's fair. The question is, is there a fundamental reason I overlooked to deny such a change? Except for positioning and whitespace, the line has been in names.

Re: [Rd] Could .Primitive("[") stop forcing R_Visible = TRUE?

2024-10-25 Thread Duncan Murdoch
My message below sounds a little aggressive, I apologize in advance for that. But I can't think of better wording. This is a small change, which will have a small benefit, mainly for the data.table authors, who really brought it on themselves by choosing to (ab)use R extraction syntax to have

Re: [Rd] Could .Primitive("[") stop forcing R_Visible = TRUE?

2024-10-25 Thread Ivan Krylov via R-devel
On Thu, 24 Oct 2024 13:23:56 -0400 Toby Hocking wrote: > The patch you are proposing to base R is > https://github.com/Rdatatable/data.table/issues/6566#issuecomment-2428912338 > right? Yes, it's this one, thank you for providing the link. Surprisingly, a very cursory check of 100 packages most