Hi Sundar,
lower-case thumbs up, with two remarks:
* the new boolean argument in NativeJavaImporter should be called isScope, not
isStrict
* ScopeTest could better illustrate the conditions it tests if it used a
symbolic constant instead of 16 ("magic number") - can the megamorphism
threshold be queried from the engine?
Best,
Michael
> Am 16.09.2015 um 06:27 schrieb Sundararajan Athijegannathan
> <[email protected]>:
>
> Please review http://cr.openjdk.java.net/~sundar/8136544/ for
> https://bugs.openjdk.java.net/browse/JDK-8136544
>
> Thanks,
> -Sundar
--
<http://www.oracle.com/>
Dr. Michael Haupt | Principal Member of Technical Staff
Phone: +49 331 200 7277 | Fax: +49 331 200 7561
Oracle Java Platform Group | LangTools Team | Nashorn
Oracle Deutschland B.V. & Co. KG, Schiffbauergasse 14 | 14467 Potsdam, Germany
<http://www.oracle.com/commitment> Oracle is committed to developing
practices and products that help protect the environment