Re: [Rd] [Bug Fix] Default values not applied to ... arguments

2017-07-07 Thread Sahil Kang
[Rd] [Bug Fix] Default values not applied to ... arguments As I said the old behavior had been for several years both in AST and BC interpreter. I fixed it in AST and I will fix it in BC. I know exactly where it is and how to fix it, there is no more help I could use on this issue and I don

Re: [Rd] [Bug Fix] Default values not applied to ... arguments

2017-07-07 Thread Sahil Kang
Yes, I see what you mean. My patch only disables JIT compilation for closures. If a user manually compiles a closure, however, the bug pops up again. I think the bug may either lie in how the byte-compiler compiles closures, or how closures with compiled body's are executed by the AST interpre

Re: [Rd] [Bug Fix] Default values not applied to ... arguments

2017-07-06 Thread Tomas Kalibera
Thanks for the report, I've fixed 15199 in the AST interpreter in 72664, I will fix it in the byte-code interpreter as well. If you ever needed to disable the JIT, there is API for that, see ?compiler. Note though that by disabling the JIT you won't disable the byte-code interpreter, because c

[Rd] [Bug Fix] Default values not applied to ... arguments

2017-07-06 Thread Sahil Kang
Hi Duncan, Martin Here's a small patch that fixes bug 15199 reported at: https://bugs.r-project.org/bugzilla3/show_bug.cgi?id=15199 I was able to reproduce the bug as Duncan had outlined just fine, but I did notice that when we debug(f), the problem went away. I later realized that f(1,,3) beha