RFR:8147076(LinkerCallSite.ARGLIMIT is used incorrectly)
Hannes Wallnöfer
hannes.wallnoefer at oracle.com
Fri Sep 29 15:39:37 UTC 2017
Looks good!
Hannes
> Am 29.09.2017 um 17:37 schrieb Srinivas Dama <srinivas.dama at oracle.com>:
>
> Hi,
>
> here is the revised patch
> http://cr.openjdk.java.net/~sdama/8147076/webrev.01/
>
> Regards,
> Srinivas
> ----- Original Message -----
> From: hannes.wallnoefer at oracle.com
> To: srinivas.dama at oracle.com
> Cc: nashorn-dev at openjdk.java.net
> Sent: Friday, September 29, 2017 7:33:18 PM GMT +05:30 Chennai, Kolkata, Mumbai, New Delhi
> Subject: Re: RFR:8147076(LinkerCallSite.ARGLIMIT is used incorrectly)
>
> Hi Srini,
>
> a few notes:
>
> - Shouldn’t the new test include an invocation with > ARGLIMIT double arguments to test it works?
> - No need to use Number(1.1), a number with a fractional part will result in a double.
> - I assume eval() in the test is to force this-object and callee to be sent, maybe add a comment about that?
> - there’s a typo in the test: arguements
>
> Hannes
>
>
>> Am 29.09.2017 um 15:34 schrieb Srinivas Dama <srinivas.dama at oracle.com>:
>>
>> Hi,
>>
>> Please review http://cr.openjdk.java.net/~sdama/8147076/webrev.00/
>> for https://bugs.openjdk.java.net/browse/JDK-8147076
>>
>> Regards,
>> Srinivas
>
More information about the nashorn-dev
mailing list