RFR 8134731: `Function.prototype.apply` interacts incorrectly with `arguments`

Marcus Lagergren lagergren at gmail.com
Tue Sep 1 10:21:41 UTC 2015


Awesome. Good diagnosis, Sundar! +1

/M

> On 01 Sep 2015, at 10:57, Sundararajan Athijegannathan <sundararajan.athijegannathan at oracle.com> wrote:
> 
> Yes. And I manually inspected generated bytecode to make sure no-declared-param function case is handled by apply2call opt.
> 
> -Sundar
> 
> On 9/1/2015 1:51 PM, Attila Szegedi wrote:
>> It can’t regress on raytrace, since raytrace uses:
>> 
>>     return function() {
>>       this.initialize.apply(this, arguments);
>>     }
>> 
>> that is, the function containing the transform declares no arguments.
>> 
>> Attila.
>> 
>>> On Sep 1, 2015, at 9:15 AM, Marcus Lagergren <lagergren at gmail.com> wrote:
>>> 
>>> +1. Any regression on raytrace, though?
>>> 
>>> /M
>>> 
>>>> On 31 Aug 2015, at 13:35, Sundararajan Athijegannathan <sundararajan.athijegannathan at oracle.com> wrote:
>>>> 
>>>> Please review http://cr.openjdk.java.net/~sundar/8134731/ for https://bugs.openjdk.java.net/browse/JDK-8134731
>>>> 
>>>> Thanks
>>>> -Sundar
> 



More information about the nashorn-dev mailing list