On Wed, 7 Oct 2020 18:09:28 GMT, Igor Ignatyev <iignat...@openjdk.org> wrote:
>> Hi all, >> >> could you please review this small cleanup which replaces >> `ManagementFactory.getRuntimeMXBean().getName().split("@")[0]` w/ >> `ProcessHandle.current().pid()` to get current >> process pid? Thanks, >> -- Igor > > Igor Ignatyev has updated the pull request incrementally with two additional > commits since the last revision: > > - use Long.toString instead of String.valueOf > - remove explicit coversion to String in Suicide.java Still looks good to me. ------------- Marked as reviewed by shade (Reviewer). PR: https://git.openjdk.java.net/jdk/pull/534