+1

Hannes

> Am 04.10.2016 um 09:51 schrieb Sundararajan Athijegannathan 
> <sundararajan.athijegannat...@oracle.com>:
> 
> +1
> 
> -Sundar
> 
> On 10/3/2016 10:47 PM, Attila Szegedi wrote:
>> Please review JDK-8167037 "Remove CALL_METHOD support from internal Nashorn 
>> linkers" at <http://cr.openjdk.java.net/~attila/8167037/webrev.jdk9> for 
>> <https://bugs.openjdk.java.net/browse/JDK-8167037>
>> 
>> Nashorn never emits CALL_METHOD Dynalink operation; therefore code for 
>> supporting it in internal linkers (those that only service call sites in 
>> Nashorn-generated bytecode) is unnecessary. 
>> 
>> I originally stumbled upon this inspecting 
>> ScriptObject.findCallMethodMethod() and noticed that the code was even 
>> somewhat incorrect by weakening the lookup of the call to public lookup. 
>> Thinking of fixing it, I realized it's actually never used.
>> 
>> Thanks,
>>  Attila.
> 

Reply via email to