Hi,

I have one more update, with a couple of suggested changes to simplify the execute() calls:

* execute() takes a vararg, so explicit String[] creation can be omitted (mostly).

* args common to every execute() call are consolidated into a List. (The resulting arg reordering should not affect test execution.)

http://cr.openjdk.java.net/~bchristi/8177328/webrev.02/

Thanks,
-Brent

On 5/10/17 9:42 AM, Brent Christian wrote:
On 5/9/17 4:39 PM, Brent Christian wrote:
On 5/9/17 4:27 PM, Mandy Chung wrote:

Webrev:
http://cr.openjdk.java.net/~bchristi/8177328/webrev.01/

Can it restore to the default timeout?

Yes.  Between the additional @runs and no longer testing automated
modules, I believe that should be fine.

Webrev updated in place, with the timeout value removed.

-Brent

Reply via email to