RFR: CODETOOLS-7902830: dtraceasm: do not Process.destroy the dtrace process [v3]
Henri Tremblay
github.com+1216819+henri-tremblay at openjdk.java.net
Tue Feb 23 14:34:46 UTC 2021
On Tue, 23 Feb 2021 11:05:04 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:
>> Look here:
>> https://mail.openjdk.java.net/pipermail/jmh-dev/2021-February/003134.html
>>
>> Process.destroy closes the streams, so whatever output the profiler had accrued would be lost.
>>
>> I need someone with functioning Mac OS dtrace to test this.
>
> Aleksey Shipilev has updated the pull request incrementally with one additional commit since the last revision:
>
> Comment updates
jmh-core/src/test/java/org/openjdk/jmh/util/TestUtil.java line 37:
> 35: @Test
> 36: public void testPID_Current() {
> 37: Assert.assertNotSame(0, Utils.getPid());
This should be an assertNotEquals no? Because assertNotSame will box the 0 and will probably always be not same no? Or an hardcoded 0 is always the same?
-------------
PR: https://git.openjdk.java.net/jmh/pull/22
More information about the jmh-dev
mailing list