+1

(with the copyright header change suggested by Alan)

-Sundar

On 30/01/18, 8:28 PM, Alan Bateman wrote:
On 30/01/2018 09:17, Srinivas Dama wrote:
Hi,

Please review the revised webrev at http://cr.openjdk.java.net/~sdama/8011697/webrev.01/ for https://bugs.openjdk.java.net/browse/JDK-8011697


The updated patch to ScriptEngineManager looks okay.

Can you replace the copyright header in the tests with the GPL version before you push this? We don't use the GPL + "Classpath" exception header in tests.

-Alan

Reply via email to