Thank you for the comments sundar.

Here is the revised patch with test case modified.
http://cr.openjdk.java.net/~sdama/8185257/webrev.01/

Regards,
Srinivas
----- Original Message -----
From: sundararajan.athijegannat...@oracle.com
To: nashorn-dev@openjdk.java.net
Sent: Tuesday, September 12, 2017 6:49:09 PM GMT +05:30 Chennai, Kolkata, 
Mumbai, New Delhi
Subject: Re: RFR: 8185257(Nashorn AST is missing nodes when a for-loop contains 
a VariableDeclarationList)

You may want to print Tree kind in the test rather than using 
implementation class name (and using that it .EXPECTED file).

Other than that, +1

-Sundar

On 12/09/17, 4:58 PM, Srinivas Dama wrote:
> Please review http://cr.openjdk.java.net/~sdama/8185257/webrev.00/
> for https://bugs.openjdk.java.net/browse/JDK-8185257
>
> Regards,
> Srinivas

Reply via email to