Looks good.
Best regards,
Vladimir Ivanov
On 6/17/15 7:23 PM, Konstantin Shefov wrote:
Vladimir,
I have updated the webrev:
http://cr.openjdk.java.net/~kshefov/8067005/webrev.01/
-Konstantin
On 06/17/2015 04:01 PM, Vladimir Ivanov wrote:
Konstantin,
I'd move isThrowableCausedByVME & processVME from Utils to a more
specific place (new shared class for JSR292 tests?). They aren't
general enough to be useful for other tests.
Also, processVME name is confusing. I'd introduce something like a
runMHTest wrapper and process exceptions there.
Best regards,
Vladimir Ivanov
On 6/17/15 1:19 PM, Konstantin Shefov wrote:
Hello,
Please review the test bug fix
https://bugs.openjdk.java.net/browse/JDK-8067005
Webrev is http://cr.openjdk.java.net/~kshefov/8067005/webrev.00/
Thanks
-Konstantin
