Looks good to me Alex. With the various refactoring that you've done, this is getting really easy to read and understand what is actually being tested :) Jc
On Thu, Aug 30, 2018 at 3:13 PM Alex Menkov <[email protected]> wrote: > Hi all, > > Please review a fix for > https://bugs.openjdk.java.net/browse/JDK-8067354 > webrev: > http://cr.openjdk.java.net/~amenkov/sh2java/GetLocalVariables4/webrev.01/ > > I was not able to reproduce original issue on Windows, but got a number > of solaris-sparcv9 failures with timeouts. > The fix converts the test from shell to java, converted test passed 400 > times w/o any issues. > > --alex > -- Thanks, Jc
