On Wed, 30 Aug 2023 11:31:20 GMT, Mark Sheppard <mshep...@openjdk.org> wrote:
> So you could create a single createJavaProcessBuilder with add an additional
> parameter boolean addTestOpts e.g.
createJavaProcessBuilder(List command, boolean addTestOpts) { ... }
@msheppar that is actually where we started, and it was then split into two
differently named methods to "make it clear" which one included the test opts
without having to remember the name of the parameter that the true/false
argument was bound to.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/15452#issuecomment-1700402301
