RFR:8147076(LinkerCallSite.ARGLIMIT is used incorrectly)

[Srinivas Dama]

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