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