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