Two suggestions:

* You may want to add a check to make sure that the control reached func.call method - there is an assert there but the test does not assert the control reached there

* class name "func" be changed to "Func" or some such (capitalization of class names)

-Sundar

On 03/11/17, 11:36 AM, Priya Lakshmi Muthuswamy wrote:
Hi,

Please review JDK-8186807: JSObject gets ScriptFunction when ScriptObjectMirror is expected

JBS: https://bugs.openjdk.java.net/browse/JDK-8186807
webrev: http://cr.openjdk.java.net/~pmuthuswamy/8186807/webrev.00/

Thanks,
Priya

Reply via email to