RFR: 8359167: Remove unused test/hotspot/jtreg/vmTestbase/nsk/share/jpda/BindServer.java
Chris Plummer
cjplummer at openjdk.org
Mon Jun 16 16:12:31 UTC 2025
On Fri, 13 Jun 2025 23:37:32 GMT, Alex Menkov <amenkov at openjdk.org> wrote:
>> The nsk BindServer class is no longer used so it can be removed. This appears to be a remnant left over after the following recent change:
>>
>> [JDK-8333117](https://bugs.openjdk.org/browse/JDK-8333117) Remove support of remote and manual debuggee launchers
>>
>> There is also some additional cleanup of "manual" and "remote" debuggee launchers done by this PR and fixing some comments that have been wrong for a long time regarding "see also".
>>
>> Tested by running nsk/jdi, nsk/jdb, and nsk/jdwp tests locally. I'll also run them with tier5 CI once reviews have been done.
>
> test/hotspot/jtreg/vmTestbase/nsk/share/jpda/DebugeeProcess.java line 44:
>
>> 42: * debugee VM process.
>> 43: * Derived classes should implement these methods corresponding to the mode
>> 44: * that the process should be started in (locally).
>
> I think this can be dropped
Which part?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/25785#discussion_r2150376249
More information about the serviceability-dev
mailing list