On Thu, 4 Feb 2021 18:25:45 GMT, Igor Ignatyev <[email protected]> wrote:

>> Roland Westrelin has updated the pull request with a new target base due to 
>> a merge or a rebase. The incremental webrev excludes the unrelated changes 
>> brought in by the merge/rebase. The pull request contains 10 additional 
>> commits since the last revision:
>> 
>>  - pid in OutputAnalyzer
>>  - pid in OutputAnalyzer
>>  - Merge branch 'master' into JDK-8260296
>>  - convert all tests
>>  - Merge branch 'master' into JDK-8260296
>>  - use CoreUtils
>>  - whitespaces
>>  - SA fixes
>>  - VM fix
>>  - test
>
> LGTM, you still need to update the copyright years as @plummercj 
> [mentioned](https://github.com/openjdk/jdk/pull/2195#issuecomment-768115732).

Yes, this is a much better solution for getting the pid.

-------------

PR: https://git.openjdk.java.net/jdk/pull/2195

Reply via email to