RFR: 8299518: HotSpotVirtualMachine shared code across different platforms
David Holmes
dholmes at openjdk.org
Wed Jan 4 02:08:50 UTC 2023
On Tue, 3 Jan 2023 09:34:55 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
There are a lot of stylistic changes here that have nothing to do with refactoring or code sharing and they make the real changes very hard to see. At a minimum things like variable renaming should be done in a separate commit, so the refactoring is more obvious, but it may be better to use a separate cleanup RFE.
src/jdk.attach/aix/classes/sun/tools/attach/VirtualMachineImpl.java line 59:
> 57:
> 58: // This provider only understands pids
> 59: int pid = Integer.parseInt(vmid);
This will now throw `NumberFormatException` instead of the expected `AttachNotSupportedException`.
-------------
PR: https://git.openjdk.org/jdk/pull/11823
More information about the serviceability-dev
mailing list