On Thu, 31 Aug 2023 05:45:27 GMT, David Holmes <[email protected]> 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.
cheers David thanks for the clarification on background
-------------
PR Comment: https://git.openjdk.org/jdk/pull/15452#issuecomment-1700749844
