Hi Max, Thanks for your review.
On 11/9/19 5:25 AM, Weijun Wang wrote: > I added my name as a CSR reviewer and your code change looks fine. > > One nit, if it were me, I would throw an exception in "CLIENT" mode and let > executeCommand() to test for .shouldHaveExitValue(0). It's up to you to > decide which is better. > I've taken this comment into Webrev.03: http://cr.openjdk.java.net/~mbalao/webrevs/8233404/8233404.webrev.03/ If there are no further objections, I'll push Webrev.03. Kind regards, Martin.-
