Looks good!
Hannes
> Am 25.08.2017 um 09:22 schrieb Srinivas Dama :
>
> Thank you Hannes for the comments.
>
> Here is the revised patch at
> http://cr.openjdk.java.net/~sdama/8184720/webrev.01/ .
>
> Note: This includes additional fix from Hannes on top of existing
Thank you Hannes for the comments.
Here is the revised patch at
http://cr.openjdk.java.net/~sdama/8184720/webrev.01/ .
Note: This includes additional fix from Hannes on top of existing fix to
resolve assertion error while
running tests from 'ant' test runner without 'scripting' option.
Hi Srini,
I don’t think the -scripting option is needed in the test, and you could just
call the function without try-catch and printing messages.
Otherwise +1
Hannes
> Am 23.08.2017 um 06:38 schrieb Srinivas Dama :
>
> Hi,
>
> Please review