+1
-Sundar
On 12/09/17, 11:02 PM, Srinivas Dama wrote:
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
