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

Reply via email to