RFR: 8127: Use ProcessHandle API to get PIDs
Christoph Langer
clanger at openjdk.org
Sun Oct 15 11:06:22 UTC 2023
On Sun, 15 Oct 2023 10:48:05 GMT, Christoph Dreis <duke at openjdk.org> wrote:
>> core/org.openjdk.jmc.common/src/main/java/org/openjdk/jmc/common/util/Environment.java line 136:
>>
>>> 134: try {
>>> 135: return (int) ProcessHandle.current().pid();
>>> 136: } catch (Exception e) {
>>
>> I think the try/catch is not needed with ProcessHandle any more...
>
> Should I remove this still?
Sure, or do you think it is needed?
-------------
PR Review Comment: https://git.openjdk.org/jmc/pull/522#discussion_r1359854907
More information about the jmc-dev
mailing list