[Rd] R_ext/Altrep.h should be more C++-friendly

2018-10-08 Thread Michael Sannella via R-devel
I am not able to #include "R_ext/Altrep.h" from a C++ file. I think it needs two changes: 1. add the same __cplusplus check as most of the other header files: #ifdef __cplusplus extern "C" { #endif ... #ifdef __cplusplus } #endif 2. change the line R_new_alt

[Rd] bug with OutDec option and deferred_string altrep object

2018-10-08 Thread Michael Sannella via R-devel
While implementing R's new 'altrep' functionality in the TERR engine, I discovered a bug in R's 'deferred_string' altrep object: it is not using the correct value of the 'OutDec' option when it expands a deferred_string. See the following example: R 3.5.1: (same results in R 3.6.0 devel engine bu

Re: [Rd] bug with OutDec option and deferred_string altrep object

2018-10-08 Thread Tierney, Luke
Thanks for the report. The approach you outlines below should work -- I'll look into it. Best, luke On Mon, 8 Oct 2018, Michael Sannella wrote: > While implementing R's new 'altrep' functionality in the TERR engine, > I discovered a bug in R's 'deferred_string' altrep object: it is not > using

Re: [Rd] Rscript -e does not accept newlines under Linux?

2018-10-08 Thread Tomas Kalibera
I've checked in an experimental fix for this (75413). The newline was lost in the shell script wrapper for R, it is now being escaped similarly to space. To pass multiple commands to Rscript, one can also use "-e" multiple times. Tomas On 09/17/2018 01:09 PM, Duncan Murdoch wrote: On 16/09/2

Re: [Rd] Warning when calling formals() for `[`.

2018-10-08 Thread Emil Bode
Hello, I agree the documentation of args can be improved, but the main question is what the return should be. I guess the reason args() returns NULL is because of the way argument-matching works for primitives: there is a lot going on under the hood, and what arguments are/are not acceptable fo

Re: [Rd] Redundant code in 'split.default' in R devel

2018-10-08 Thread Martin Maechler
> Suharto Anggono via R-devel > on Fri, 5 Oct 2018 17:13:30 + writes: > After r75387, function 'split.default' in R devel still has this part that no > longer has effect. > lf <- levels(f) > y <- vector("list", length(lf)) > names(y) <- lf Indeed --> removed now.