RFR: 8299518: HotSpotVirtualMachine shared code across different platforms [v6]

Chris Plummer cjplummer at openjdk.org
Wed Jan 11 19:16:26 UTC 2023


On Tue, 10 Jan 2023 10:49:52 GMT, Yi Yang <yyang at openjdk.org> wrote:

>> harmless refactor to share code across different platforms of VirtualMachineImpl:
>> 1. Shared code to process command response after requesting a command execution
>> 2. Read functionality in SocketInputStream can be reused
>
> Yi Yang has updated the pull request incrementally with three additional commits since the last revision:
> 
>  - format
>  - Merge branch 'jdk_virtualmachienimpl' of github.com:y1yang0/jdk into jdk_virtualmachienimpl
>  - -1 to vmid

src/jdk.attach/share/classes/sun/tools/attach/HotSpotVirtualMachine.java line 455:

> 453:             if (fd != -1) {
> 454:                 close(fd);
> 455:             }

There used to be logic to set `fd` (previously called `s`) to -1 during the close operation. Is that no longer needed?

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

PR: https://git.openjdk.org/jdk/pull/11823


More information about the serviceability-dev mailing list