RFR: 8315097: Rename createJavaProcessBuilder [v6]
Leonid Mesnik
lmesnik at openjdk.org
Tue Oct 24 17:04:41 UTC 2023
On Tue, 24 Oct 2023 07:49:30 GMT, Leo Korinth <lkorinth at openjdk.org> wrote:
>> This pull request renames `createJavaProcessBuilder` to `createLimitedTestJavaProcessBuilder` and renames `createTestJvm` to `createTestJavaProcessBuilder`. Both are implemented through a private `createJavaProcessBuilder`. It also updates the java doc.
>>
>> This is so that it should be harder to by mistake use `createLimitedTestJavaProcessBuilder` that is problematic because it will not forward JVM flags to the tested JVM.
>
> Leo Korinth has updated the pull request incrementally with six additional commits since the last revision:
>
> - static import
> - copyright
> - whitespace
> - whitespace
> - sed
> - fix test/lib/jdk/test/lib/process/ProcessTools.java
Marked as reviewed by lmesnik (Reviewer).
-------------
PR Review: https://git.openjdk.org/jdk/pull/15452#pullrequestreview-1695447658
More information about the nio-dev
mailing list